One more question - is the text "Alerts" hardcoded? I could not find it in language files and was not able to locate template where to change it as well?
![[Image: attachment.php?aid=27315]](https://camo.mybb.com/0cea6f284d3d82b4d1b8ca75c62fc9519c2f4ce7/687474703a2f2f636f6d6d756e6974792e6d7962622e636f6d2f6174746163686d656e742e7068703f6169643d3237333135)
Found it!
Found it!
MyAlerts - a notification/alert system for MyBB
|
One more question - is the text "Alerts" hardcoded? I could not find it in language files and was not able to locate template where to change it as well?
Found it!
2012-09-27, 04:54 PM
![]()
2012-09-27, 06:32 PM
I can do that, yes. Nothing is cached as caching should only be used for objects that don't frequently change. Obviously, alerts do change fairly often. Templates are, of course, cached though.
Ok, global queries: 1;
User CP (Alerts listing): 12 total; User CP (Alerts settings): 9 total; Sending PM: 1; Adding to buddylist: 1; Adding rep: 1; Quoting: 2 (though one of them can be sizeable depending on how many people are getting quoted...) Reply to thread: 1; I've tried to keep them to an absolute minimum wherever I can. There are likely more optimisations I can make and that's pretty much all I'm working on right now.
2012-09-27, 06:57 PM
Quote:SELECT COUNT(id) AS count FROM mybb_alerts WHERE uid = 1 AND unread = 1 AND type IN ('rep','pm','buddylist','quoted','thread_reply') IMO you should just add a new user column to save the number of alerts pending reading. That will remove one query for each page being seen.
2012-09-27, 07:01 PM
That seems like such an untidy way of doing things. I already hate how I store alerts settings. If you've taken any classes in database structure you'll know why I'm against it
![]()
2012-09-27, 08:28 PM
Oh well, I have not even read a single PHP/DB book, lol
It just seems to me that it is better that reading a full DB table each time a page is loaded.
2012-09-27, 08:57 PM
It may well be "better" in that way, but it means adding a query each time an alert is added and storing a count of values within another table in a column is really not a good idea
![]()
2012-09-28, 06:03 PM
Has there been any progress on getting Frost to update his plugin?
2012-09-28, 06:15 PM
He has updated his PluginLibrary and MyAlerts seem to work with it
![]()
Support PMs will be ignored!
|
« Next Oldest | Next Newest »
|