Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Easier and centralised way to handle numbers next to menu items #695

Merged
merged 2 commits into from Jul 20, 2013

Conversation

emanuele45
Copy link
Contributor

Signed-off-by: emanuele emanuele45@gmail.com

Signed-off-by: emanuele <emanuele45@gmail.com>
Signed-off-by: emanuele <emanuele45@gmail.com>
@emanuele45 emanuele45 mentioned this pull request Jul 19, 2013
norv added a commit that referenced this pull request Jul 20, 2013
Easier and centralised way to handle numbers next to menu items
@norv norv merged commit 5626928 into elkarte:master Jul 20, 2013
@norv
Copy link
Contributor

norv commented Jul 20, 2013

I merged this to take advantage of the fixes now. (too many errors!)

@emanuele45 emanuele45 mentioned this pull request Jul 20, 2013
@Spuds
Copy link
Contributor

Spuds commented Jul 20, 2013

Never mind it seems !

@emanuele45 emanuele45 deleted the fix_menu branch July 20, 2013 12:09
@norv
Copy link
Contributor

norv commented Jul 21, 2013

Looking at these it seems that the cache on the totals is very needed, the one on the menu seems less relevant.

Please keep in mind that moderation as it is today, is quite limited. When we will want more features to it (moderator groups, some request on follow-ups, etc), a cache in session will help more. Perhaps we could (i.e. next time we'd have a proposal for moderation features) look into extending the mod cache, and use it for any extra info needed. Let me know if I'm missing the point of the discussion here; I think this was the reason for the mod cache in the first place. (and not as much the numbers alone, although on those I'd say it proves its value.)

@emanuele45
Copy link
Contributor Author

Yeah, I think you are talking about another cache... ;)

We are talking just about the "normal" cache_get_data and cache_put_data. :)

@norv
Copy link
Contributor

norv commented Jul 21, 2013

Ignore it then. I read too fast, bad things can happen when I do that!

Just a few quick thoughts on caching the menu: there must be many keys not matching for users other than regular users and guests, even if the menu will be the same; not all groups are relevant for menu, possibly by far. OTOH, boardsAllowedTo() is called by loadModeratorMenuCounts().

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.

None yet

3 participants