Event Calendar v3

[DivaVocals]

My post expired so I wanted to add this.. Also I need to explain why I am advocating for admin settings versus having novice shopowners edit PHP files.. I know from experience of being a member of this community that most novice web designers and shopowners often run afoul of things when they make edits in PHP files. Putting admin and configuration items into a settings menu minimizes this possibility as fewer problems are seen when these kinds of configuration/options are properly put into an admin setting menu. The menu provides shopwoners easy access to set things up the way they want, and they do not have to monkey with the code to get there.. Options like day long versus short months should be made an admin option that is accessed via an admin menu so that novice shopowners will have fewer issues installing and setting this module up.

[DivaVocals]

Wait, here's the new files....I put in two options, one for dropdowns and 1 for titles...this also has the mod so the month/year stay selected throughout the users session:
Attachment 12361

Here's the fix for the image manipulator, just copy over the old in your_admin/includes/classes/
Attachment 12362

Thanks,
Rus

Cool.. I'll incorporate this into the my working files and I'll update the admin settings menu script to add the new options.. I may even go an extra step to include an auto-installer file for the admin menu script to further reduce installation errors (this is especially helpful for novice shopowners)

[DivaVocals]

Something to add to our list:
The Calendar class is very inefficient. It hits the sql server for every single day while building the monthly calendars...that's a minimum of 28 hits to the db every time you update your page (if using the side box calendar). When you click on a year that's 390+ hits to the db. It should hit it once per month calendar at most and then once when looking at the year. Something to keep in mind.

Didn't want this to get lost because you make a VERY good point.. Is this something that you are working on for this module or is it something you are raising that should be put on a to-do later list??

[LaCamus]

Didn't want this to get lost because you make a VERY good point.. Is this something that you are working on for this module or is it something you are raising that should be put on a to-do later list??

We should probably pause here and regroup...get all the mods put together so everyone can start working off the same version.

[DivaVocals]

We should probably pause here and regroup...get all the mods put together so everyone can start working off the same version.

I agree..

[LaCamus]

I agree..

Dang it, all right, you convinced me. I'll optimize the Calendar class, do the mini file handler, attempt to separate the code from tpl_main_page.php... and then pause.

[DivaVocals]

Dang it, all right, you convinced me. I'll optimize the Calendar class, do the mini file handler, attempt to separate the code from tpl_main_page.php... and then pause.

We finally GOTCHA!!!

[dw08gm]

Wait, here's the new files....I put in two options, one for dropdowns and 1 for titles...this also has the mod so the month/year stay selected throughout the users session:
Attachment 12361

Here's the fix for the image manipulator, just copy over the old in your_admin/includes/classes/
Attachment 12362

Thanks,
Rus

Brilliant, will check these tonight. Despite our time differences, I sense the events_calendar camel is almost in train.

Cheers

[dw08gm]

Issues considered fixed:

Admin Side:
GIF/PNG transparencies now preserved upon resizing these types of transparent images in event manager.

Sidebox
Month/Year dropdown selections now retained through subsequent page refreshing or link clicking, and can now be used for greater calendar control (selecting specific calendar months/years, calendar years, and for easily returning to view multiple events within the aforesaid selections).


Except for one minor problem:

Store Side:
Because images larger than the encapsulating divs would not automatically shrink to within the encapsulating divs, nor shrink in proportion, I had to change Line 205 in \includes\templates\amanita\templates\tpl_events_calendar_default.php
from

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1" hspace="0" vspace="0"', true); ?>

to

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1" hspace="0" vspace="0"', false); ?>

while the following also worked

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1"', false); ?>

I found it made no difference toggling through the following Admin > Images settings:
Calculate Image Size = True (default)
Image - Use Proportional Images on Products and Categories = 1 (default)

or switching zen_image_OLD with zen_image in \includes\functions\extra_functions\events_calendar_functions.php (Line 70).

$dumi = ($usew ? zen_image_OLD($wf, $alt, '', '', $params) : ('<img src="'.$wf.'" alt="'.$alt.'" title=" '.$alt.' " '.$params.' />'));

While I remain unclear what that true/false setting in tpl_events_calendar_default.php is intended to do exactly, I have not yet found any other way to fix this problem.

As I contemplate re-writing the readme, I am wondering how to explain this setting, and whether or not the changing of this setting can or should be placed in a language file or elsewhere.

Has anyone else experienced the same or similar problem with images larger than the encapsulating divs?



As I consider all known bugs have now been fixed or otherwise resolved, and notwithstanding the minor problem raised above, further problems, comments, criticisms or objections, I would be immensely pleased to undertake the final packaging of this mod for submission to Plugins beginning this weekend.

Rus and Diva, I cannot thank you enough for your fabulous contributions and untiring efforts, other than to say brilliant, absolutely brilliant, to the both of you.

Cheers

[DivaVocals]

Store Side:
Because images larger than the encapsulating divs would not automatically shrink to within the encapsulating divs, nor shrink in proportion, I had to change Line 205 in \includes\templates\amanita\templates\tpl_events_calendar_default.php
from

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1" hspace="0" vspace="0"', true); ?>

to

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1" hspace="0" vspace="0"', false); ?>

while the following also worked

<?php echo event_flink_small($event_array['image'], $event_array['title'], 'class="c1"', false); ?>

I found it made no difference toggling through the following Admin > Images settings:
Calculate Image Size = True (default)
Image - Use Proportional Images on Products and Categories = 1 (default)

or switching zen_image_OLD with zen_image in \includes\functions\extra_functions\events_calendar_functions.php (Line 70).

$dumi = ($usew ? zen_image_OLD($wf, $alt, '', '', $params) : ('<img src="'.$wf.'" alt="'.$alt.'" title=" '.$alt.' " '.$params.' />'));

While I remain unclear what that true/false setting in tpl_events_calendar_default.php is intended to do exactly, I have not yet found any other way to fix this problem.

As I contemplate re-writing the readme, I am wondering how to explain this setting, and whether or not the changing of this setting can or should be placed in a language file or elsewhere.

If you refer back to some of my recent posts this is explained.. "true" adds the "height" and "width" attributes to the IMG tag, false or NO value leaves them out.

so you either end up with HTML that looks like this:

<imgsrc="images/events_images/IMAG0060.jpg" alt="Recurring Event" title=" Recurring Event " width="600" height="450" class="event_calendar_image" />

or it looks like this:

<imgsrc="images/events_images/IMAG0060.jpg" alt="Recurring Event" title=" Recurring Event " class="event_calendar_image" />

If you look at the zip files I uploaded, and my previous posts on this, this is RESOLVED as I created an an admin setting so that EITHER option can be used without having to manually edit any file. There should be NOTHING to explain in the readme about HOW to change the setting other than to direct them to the admin setting.