Anmari forum

for wordpress plugins and web matters

1. 

   watermelonwebworks
   Member

   Good Evening Anmari,

   We recently purchased the amr-events plugin as well as downloaded the free events list plugin and we are doing the initial setup right now. We do have a php error throwing where I have isolated an if statement that might have an issue.

   Here's our system environment:
   Wordpress: 3.4.2
   Network/Multisite
   amr events: 3.0.10
   amr events calendar or lists with ical files: 4.0.30

   We have a multisite and we have network installed both AMR Events as well as the events list plugin. When an event was made, we got the following error persistently:

   Notice: Undefined index: type in /home/ourhost/public_html/wp-content/plugins/amr-ical-events-list/includes/amr-pluggable.php on line 753

   I have commented out the if statement for line 753-759, and this has stopped the error from generating. Here's the exact chunk of code:

   if (($e['type'] == 'VFREEBUSY') ) {
   $e['excerpt'] = $e['SUMMARY'];
   $e['DESCRIPTION'] = $e['SUMMARY'];
   if (!empty($amr_options['freebusymessage']))
   $e['SUMMARY'] = $amr_options['freebusymessage'];
   }

   Is this chunk of code critical to the plugin or could it be commented out until the next update? Thanks in advance.

   Posted 6 months ago #
2. 

   anmari
   Key Master

   Hi watermelonworks (love the name!),

   It's a 'notice' not an error, so should only be showing up in a development system with all debug notices switched on. However I do try to avoid having notices - bad form, better to deal with them specifically.

   Commenting it out will not hurt if you are not using ics files with freebusy components.

   A better correction would be:
   

   if (isset($e['type']) and ($e['type'] == 'VFREEBUSY') ) {

   that way the code will deal better with any 'freebusy' items that may one get included (either via ics file or possibly in amr-events).

   Thank you for letting me know - the update above will be in the next version update.

   Posted 6 months ago #

RSS feed for this topic

Reply

You must log in to post.