• Resolved richrabago

    (@richrabago)


    Hi,

    After installing the “Outlook Export Buttons” extension I am getting an error on every event page.

    Parse error: syntax error, unexpected end of file, expecting elseif (T_ELSEIF) or else (T_ELSE) or endif (T_ENDIF) in H:mypath-hidden-from-post\wp-content\plugins\tec-labs-outlook-export-buttons\src\views\blocks\event-links.php on line 93
    There has been a critical error on this website.

    Learn more about troubleshooting WordPress.

    I am using Version 1.1.1 of the extension and I am using Version 5.13.0 of the Events Calendar.
    @aguseo I am not sure if this relates to the fix you did on Jan 24.

    Any help would be greatly appreciated.

    Thank you!
    Rich

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Contributor Andras Guseo

    (@aguseo)

    Hi @richrabago

    I will try to check.

    Questions:
    Are you using the classic editor or the block editor?
    Is this also happening when you only have TEC and the extension active with your theme?
    Same as above with a default theme?

    Thanks,
    Andras

    Thread Starter richrabago

    (@richrabago)

    Hi @aguseo

    Thank you so much for the fast response. I am using the new blocks editor. I will admit I was very afraid of swapping out my theme and de-activating my plugins. I know I should be able to do that freely, but I am newer to wordpress and just a bit skittish. I would be in big trouble if I broke the site. I will say I have several memberpress plug-ins and running divi (child them). I also first installed the latest version TEC (Version 5.14.0.1).

    So, instead of that testing, I took at the code in the file listed in the error posted above. Through trial and error of commenting out pieces I was able to discover what the problem is (at least for me).

    At the end of the first two if statements the closing endifs did not have ?php. When I add the php it starts to work. Not sure if that is due to the version of php I am running or some strict type setting. I am new-ish to php so I may not have it right. But it works.

    In my file the line numbers are 49 and 66. The original code is <? endif; ?> and I changed it to <?php endif; ?> .

    And in github the lines are:
    https://github.com/mt-support/tec-labs-outlook-export-buttons/blob/3ce7b3b23a1ebfb2f165590c5d2825812fc94d84/src/views/blocks/event-links.php#L49
    https://github.com/mt-support/tec-labs-outlook-export-buttons/blob/3ce7b3b23a1ebfb2f165590c5d2825812fc94d84/src/views/blocks/event-links.php#L66

    I tried to override the file in my child theme but it would not take and I kept getting the same error.

    Here is where I tried to override the file. Did I not do that correctly?
    circlemodernjuris\wp-content\themes\Divi-Child-Theme\tribe\events\blocks\event-links.php

    Therefore, I just made the above fix in the file located at:
    circlemodernjuris\wp-content\themes\Divi-Child-Theme\tribe\events\blocks\event-links.php

    Please let me know if my fix is the real issue or I am missing something.

    Again, thank you for your fast response. And thank you for The Events Calendar – it really is a great plugin.
    Rich

    Plugin Contributor Andras Guseo

    (@aguseo)

    @richrabago

    Sorry, was on vacation last week and I’m catching up now.

    Ah crap, those missing <?php tags are my fault. So if that fixes it for you, then all good.

    I will push a fix for that now and will let you know when it’s out.

    Let me know if there is anything else.

    Cheers,
    Andras

    Plugin Contributor Andras Guseo

    (@aguseo)

    @richrabago

    Could you please pull a fresh version (1.1.2) from here and check if that works for you?
    https://github.com/mt-support/tec-labs-outlook-export-buttons/archive/refs/heads/main.zip

    Thanks,
    Andras

    Thread Starter richrabago

    (@richrabago)

    Hi @aguseo

    Thank you for the the response and fix. I installed version 1.1.2 of the outlook buttons plugin from your link above and it works. So, all is good for me on this plugin. Please let me know if you make any other changes to version 1.1.2 so I can get it from the site and install it.

    The only thing I still can’t figure out is why my attempt to override the the file in my child theme did not work. Here is where I put the overridden file (circlemodernjuris\wp-content\themes\Divi-Child-Theme\tribe\events\blocks\event-links.php). But this could be a PEBKAC problem.

    Thank you again very much!
    Rich

    Hi Rich!

    Any subsequent changes to the extension will have a new version number.

    As far as why your template override isn’t working, I found that the extension is setting the priority of its custom template in such a way that it blocks the template added to your child theme from being used.

    I was able to come up with a snippet to adjust the priority, giving precedence to the file in your theme instead.

    add_filter( 'tribe_template_path_list', function( $folders ) {
    	$folders['tec-labs-outlook-export-buttons']['priority'] = 100;
    	return $folders;
    }, 100, 2 );

    Let me know if that works for you!

    Best,
    Sky

    Thread Starter richrabago

    (@richrabago)

    Hi @skyshab,
    Thank you so much for following up. I am new to wordpress and so wasn’t sure if it was my mistake. I will try out your suggestion and will post back here if it works.

    Thanks again,
    Rich

    Hi again,

    OK! Let me know if the snippet helps, once you have a chance.

    Best,
    Sky

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘Outlook Export Buttons’ is closed to new replies.