OUGC Pages Review

[Sama34]

Quote:It's probably not clear what one needs to exactly do to integrate Pages into the forum.  By default, no links are added.

There I'm once again assuming things, since I do use the Global Menu plugin by RateU to manage my own links and I expect some minimum understanding on the template system and therefore HTML I never though about this. I will probably add a feature to do this less manually.

Quote:The attempt to filter out categories doesn't make a lot of sense, since the rest of the code seems to assume that all pages are stored in the cache.

Hidden categories should be not visible nor its pages, that is why I only intend to cache visible pages within visible categories. Though yeah the implode fortunately seems to be wrong because from a quick look the query should be an 'IN' not 'NOT IN', haha..

And yeah, the categories cache is not used even when that was the plan. Unsure about the reason though.

Quote:my guess is that it's just a generic include across all OUGC plugins?

Correct. I will need to update all of my plugins that use that function at some point. All of them use the same named function...

Quote: personally don't agree with stuffing all the code into the ougc_pages.php file.  For example, the ougc_pages_show function is only ever called from pages.php, so it may as well be placed there so that it doesn't need to be loaded on every page load.

This was to return a value instead of directly printing the page to use within the portal. I removed this functionality before releasing because of lack of time for testing it. Commented code can be seen within all the script.

Thanks for your review Zinga Burga.