Skip to content

Conversation

@mbrodala
Copy link
Contributor

This ensures the colPos field is also filtered for IRRE children.

@IchHabRecht
Copy link
Owner

I have to think and try some known IRRE constellations. I'm quite sure there was an issue why I excluded IRRE elements from being processed.

@mbrodala
Copy link
Contributor Author

OK, let me know if you need some testing.

@josefglatz
Copy link
Contributor

I actually have troubles with IRRE childrens:

My scenario: confusing restrictions in a colPos:

Starting point

With the following colPos config, no CType miniFactox, miniYoutube or themeproject_sidebarEvents is allowed

	allowed {
		CType = list, miniFactox, miniYoutube, themeproject_sidebarEvents
		list_type = articles_articles
	}

Confusing Workaround

If I remove allowed.list_type completely and add all others except list_type articles_articles to disallowed.list_type all work like it should. The following config allows using a CType like themeproject_sidebarEvents within the colPos

	allowed {
		CType = list, miniFactox, miniYoutube, themeproject_sidebarEvents
#		list_type = articles_articles
	}
	disallowed {
		list_type = infogram_show
	}

Has this something to do with this issue?

@IchHabRecht
Copy link
Owner

@josefglatz your problem should be solved with an empty item in allowed.list_type= ,articles_articles

For CType other than list the list_types stays empty. That is why you have to allow an empty value as well. Currently only articles_articles is allowed which means you have to switch to that list_type before any other CType can be used.

@josefglatz
Copy link
Contributor

josefglatz commented Nov 13, 2017

@IchHabRecht I see. Is that something should be mentioned in the README.md? Btw that does the trick :-) Many thanks!!!

This ensures the colPos field is also filtered for IRRE children.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants