ezsystems/eztags — pull request #52 by bchoquet-heliopsis (2012-06-14 17:03:17+00:00)
Title: Hide tags (multilanguage version)

Hi,
here I come again with my bloody feature ;)

I took my code from pull request #39 and revamped it for version 2. I guess it'll be easier for you to review it this way and add it to the next release instead of creating an intermediary v1.

Hidden tags are now filtered only in the low level `eZtagsObject::fetchCustomCondsSQL` method.

I began to work on passing an ignoreVisibility flag from method to method but it quickly became messy and created a few logic problems (for example, what to do when getting parent attribute of an invisible tag object ?).

Do you think that flag is really necessary ?

You can check it out on https://github.com/Heliopsis-HQ/eztags/commit/3c92ac061e7ae520a10d969c3cfff20ca6da31ba

Thanks

Ben