|
d37toastmasters
|
February 7, 2014 at 1:58 am
Hi,
I’m experiencing 3 issues with the Calendar Table Add-On with the issue #1 being a showstopper. The webpage using the plugin can be found here: http://d37toastmasters.org/district-calendar-table/
1) All events requiring member registration are replaced with “You are not logged in…..” If you have 10 events requiring member registration then you have this text repeating 10 times. I’m suggesting this text is displayed only after user selects “Register Now” button / image. In other words, this should work the same way as the calendar preview does. Otherwise this breaks the table displayed on the page and makes no sense to the end user.
2) US Zip Code is missing in “Where:” line under each event.
3) For events not requiring the registration the “Register Now” button should be replaced with “View Now” button. In other words, this should work the same way as the calendar preview does.
Thank you,
Jake
|
|
Dean
|
February 7, 2014 at 3:42 am
Hi Jake,
1) I’ll make a ticket to discuss whether we should remove that, but I agree with you that it looks weird to have that showing like that.
To resolve it in the mean time you can comment out
if ( function_exists('espresso_members_installed') && espresso_members_installed() == true && !is_user_logged_in() && ($member_only == 'Y' || $member_options['member_only_all'] == 'Y') ) {
event_espresso_user_login();
}else{
and
}// close is_user_logged_in
2) The postal code/zip code was intended to be left out, it was designed to be a quick overview of the address so just first line, city and state/region.
You could add it in by modifying the template approx line 134 to state:
<?php echo stripslashes_deep($event->venue_address.', '.$event->venue_city.', '.$event->venue_state . ', ' . $event->venue_zip); ?><br />
3) This is on the calendar? Perhaps I have missed something in Beta but on the latest release ver (2.2.2) it states Register Now even for member only events. I’ll still make a ticket to look into that, but cannot confirm if/when it will be added.
|
|
d37toastmasters
|
February 8, 2014 at 4:23 am
Hi Dean,
Thank you for your reply. What is the location/file name for changes described in 1) and 2) ?
For 3) events with “Display registration form? = Yes” the button / image should read “Register Now”. For events with “Display registration form? = No” the button / image should read “View Details”. This is exactly the behavior of Calendar plugin for hover-overs.
Thank you,
Jake
|
|
Sidney Harrell
|
February 8, 2014 at 4:22 pm
For #1 it should be in plugins/espresso-template-calendar-table/index.php on line 107 to 109 and line 141. For #2 it should be the same file, line 134.
For #3, you could replace line 67 in plugins/espresso-template-calendar-table/index.php with:
$regButtonText = $event->display_reg_form == 'Y' ? '<img class="buytix_button" src="'.WP_PLUGIN_URL. "/".plugin_basename(dirname(__FILE__)) .'/register-now.png" alt="Buy Tickets">' : __('View Details', 'event_espresso');
$live_button = '<a id="a_register_link-'.$event->id.'" href="'.$registration_url.'">'.$regButtonText.'</a>';
It’s not perfect. You’d need to create a corresponding ‘View Details’ button image, like there is for the register-now button. I’ll see if we can find which member of the team made those originally and see if we can get a ‘View Details’ button made.
|
|
d37toastmasters
|
February 9, 2014 at 2:45 am
Hi Sidney,
Thank for you for the additional information on the file locations. I made the changes for #1, 2 and 3.
#1 and #2 are working now exactly as I hoped, thank you !!!
Under #3 I just made links instead of buttons. It is OK for now, however my theme supports buttons: http://theme-fusion.com/avada/buttons-social-icons/
Would it be possible to use small blue button (a.button.small.blue) in place of Register Now and View Details links? If yes, what would the code look like on line 67 in in plugins/espresso-template-calendar-table/index.php?
Thank you,
Jake
|
|
Dean
|
February 10, 2014 at 1:17 am
Hi Jake,
Yes it is possible, here’s an example from line 67 in the template index.php
$live_button = '<a class="shortcode button yellow" id="a_register_link-'.$event->id.'" href="'.$registration_url.'">BUY NOW</a>';
What I have done is removed the image and replaced it with text and then added the CSS classes to the link.
Your system may vary slightly but this worked on my theme http://take.ms/I8LwG
You would need to do this for each button type (registers, closed, etc)
|
|
d37toastmasters
|
February 13, 2014 at 3:31 am
Hi Dean,
Thank you for your reply. Now, I have have Register buttons associated with every event: http://d37toastmasters.org/district-calendar-table/
I’d like to revert to Sidney’s idea posted above and display Register button only for events requiring registration and display View button for events that do not require registration form. What changes do we need to make the index.php file? My current code is:
$regButtonText = $event->display_reg_form == ‘Y’ ? __(‘Register’, ‘event_espresso’) : __(‘View’, ‘event_espresso’);
$live_button = ‘id.'” href=”‘.$registration_url.'”>Register‘;
|
|
Dean
|
February 13, 2014 at 5:31 am
Hey,
Well if you just require text, you could add something like this at about about line 86
$regButtonText = $event->display_reg_form == 'Y' ? __('Register', 'event_espresso') : __('View', 'event_espresso');
$live_button = '<a class="shortcode button yellow" id="a_register_link-'.$event->id.'" href="'.$registration_url.'">' . $regButtonText . '</a>';
It would look something like this http://take.ms/sxpq3 (I have styled the links to appear as buttons)
|
|
d37toastmasters
|
February 13, 2014 at 5:44 am
Dean,
Thank you. That did the trick!
Jake
|
