Update:
Removed the following define from includes/languages/english/events_calendar.php
Reason: This is a duplicate of a an existing default Zen Cart admin configuration setting and this define is used ALL OVER Zen Cart. There is no reason to keep this define setting as it is already controlled in the admin settings. I suspect this is leftovers from this modules Zen Cart 1.3.7 roots that never got pulled out as it should have been.Code:define('MAX_DISPLAY_PAGE_LINKS', 5);
Removed the following define from includes/languages/english/events_calendar.php
Reason: This define is not being used ANYWHERE in the Events Calendar module. Again, I suspect this is leftovers from this modules Zen Cart 1.3.7 roots that never got pulled out as it should have been.Code:define('TEXT_DISPLAY_NUMBER_OF_EVENTS', 'Displaying event nB0 <b>%d</b><b>%_</b> of <b>%d</b>');
I modified the following from includes/languages/english/events_calendar.php
to readCode:define('TEXT_DISPLAY_NUMBER_OF_PAGES', 'Displaying <b>%d</b> to <b>%d</b> (of <b>%d</b> events) ');
Reason: The TEXT_DISPLAY_NUMBER_OF_PAGES define is a duplicate of another existing Zen Cart define for the EZ Pages. (YOUR_ADMIN/includes/languages/english/ezpages.php) Though this doesn't appear to be problematic, to avoid ANY possible issues, it makes sense that the Events Calendar module should use it's OWN unique define.Code:define('TEXT_DISPLAY_NUMBER_OF_EVENTS', 'Displaying <b>%d</b> to <b>%d</b> (of <b>%d</b> events) ');
In support of this I edited: includes/modules/events_calendar_listing.php and replaced all instances of TEXT_DISPLAY_NUMBER_OF_PAGES with TEXT_DISPLAY_NUMBER_OF_EVENTS
I'm making on more pass through all the language defines to make sure that I've moved out all of the defines that should be in the admin settings menu, and still doing more regression testing.. But I think we are really nearly here..
I do have one question.. Did we NOT want a start and end TIME for events???



