I've created a new feature following feedback last weekend. Was a quick and simple change considering the value I think it will give you as a user of BM...
Aww, you made a mess of the calendar!! I'm reporting you!!
Appropriate considering I'm brushing up on my Maths, I'm not keen on seeing stars tho lol.
I would have preferred the stars to appear on the favourites page.
JP, you're right - I forgot to exclude disabled accounts. I'll fix it now.
Sandi, you're also right - I don't like the look of the stars but I can't think of a better way to highlight the event without creating more messy colours (the weekends already have another colour and the code is getting really complicated)
Matt, you've broken it!
Microsoft OLE DB Provider for SQL Server error ?e14'
Incorrect syntax near '*'.
/sites/bm/events/calendar.asp, line 236
Error message appeared when I clicked on the calendar and not all the months were showing.
It's ok now.
Without massively overhauling the calendar page, which I can't do right now, the performance of the calendar page is really bad once I make it check for disabled users.
So, we either have the page taking 20 seconds to open if it has to check which attendees are disabled, or we have a quick count (which includes disabled accounts) and it only takes 2 seconds max.
The problem is with the way or originally designed the calendar... it basically runs a check against the database for every day on the page. If I was to design it now I would create one big database query which loads the whole page's data (thus 90x faster by loading one time and not 90 times).
Obviously I'm not going to spend days redesigning it right now but might consider it for the future.
I think for the moment we will have to settle with the attendees count including disabled accounts in view of the massive performance hit the server takes otherwise