X
31628

Ticket #31628 Public

EventON - Event Tickets and PHP v7.0.x compatibility report error(s).

  • We’re in the process of converting to PHP 7.0.x and discovered the following results during our compatibility checks, specific to EventON – Event Tickets:

    /eventon-tickets/includes/lib/src/Adapter/CPDF.php
    —————————————————————————————————–
    FOUND 1 ERROR AFFECTING 1 LINE
    —————————————————————————————————–
    190 | ERROR | Extension ‘cpdf’ is removed since PHP 5.1 – use pecl/pdflib instead.
    —————————————————————————————————–

    /eventon-tickets/includes/lib/src/FrameDecorator/Table.php
    ————————————————————————————————————————-
    FOUND 1 ERROR AFFECTING 1 LINE
    ————————————————————————————————————————-
    201 | ERROR | Using ‘break’ outside of a loop or switch structure is invalid and will throw a fatal error since PHP 7.0
    ————————————————————————————————————————-

    Please advise.

    Thanks!

    BY: VPREATR - Mar 19,2017 AT 1:38PM - 3 years ago
    • By: Ashish Rai Mar 21,2017 AT 12:34AM - Posted 3 years ago - #31757

      Thank you for your messages, I am going to assign this ticket to Ashan and he will be able to take it from here and find you a solution. Please allow some time for him to get back to you, we greatly appreciate your patience and thank you for being a eventon customer!

    • By: Ashan Perera Mar 21,2017 AT 9:52AM - Posted 3 years ago - #31814

      Yea we are not using that library any more in the new version which we will release this week hopfully. Been ready to go out and then we find issues and then take more time testing and fixing those 🙂

    • By: VPREATR Mar 26,2017 AT 8:22AM - Posted 3 years ago - #32201

      Hi Ashan,

      We’ve updated to the latest version of EventON Tickets v1.5 and in follow-up, performed an additional compatbility test(s) for PHP 7.0; this was the most recent result:

      FILE: /eventon-tickets/includes/lib/src/Adapter/CPDF.php
      —————————————————————————————————–
      FOUND 1 ERROR AFFECTING 1 LINE
      —————————————————————————————————–
       190 | ERROR | Extension ‘cpdf’ is removed since PHP 5.1 – use pecl/pdflib instead.
      —————————————————————————————————–

      FILE: /eventon-tickets/includes/lib/src/FrameDecorator/Table.php
      ————————————————————————————————————————-
      FOUND 1 ERROR AFFECTING 1 LINE
      ————————————————————————————————————————-
       201 | ERROR | Using ‘break’ outside of a loop or switch structure is invalid and will throw a fatal error since PHP 7.0
      ————————————————————————————————————————-

      We’re ready to throw the switch to so speak on our PHP 7.0 change over primarly due to performance issues, although I won’t take any chances with breaking such a critical plugin.

      Thoughts?

    • By: VPREATR Mar 26,2017 AT 10:39PM - Posted 3 years ago - #32219

      Hi Ashan,
      Interestingly enough, my EventON Tickets installation has remnants of previous versions, specifically the folder ‘/includes/lib/src‘ and ‘/includes/lib/src/lib‘.  It appears that the most recent versions don’t include any reference to those folders, so I’ve since removed the unused remnants.

      As such, yes, your reply was correct in that you don’t use that library any longer; it’s just that if someone upgrades manually and doesn’t replace the entire plugin folder, it can retain remnants and the PHP 7.0 compatibility test will attempt to analyze accordingly.

      That should resolve the any further issues during our compatibility testing.

      Thanks!

    • By: Ashan Perera Mar 27,2017 AT 9:33AM - Posted 3 years ago - #32281

      thanks for letting me know. The user updating have to be aware of this as there is no programatic way to remove those folders. But we will make a note of this.

      Glad the PHP issues are resolved for you.

You must login to reply to this ticket

HelpDesk

Welcome to EventON helpdesk.

EventON is the #1 Best selling event calendar plugin for WordPress websites in envato marketplace.

Checkout EventON