Serious malfunction –

  • Resolved WP Apps

    (@wp-apps)


    7 years, 7 months ago

    The developers of “The Events Calendar” does not appreciate WordPress standards, which causes on some cases very serious issues, especially when you are trying to synchronise your events with other systems.

    WordPress uses UTF-8 character set. The “The Events Calendar” developers are relaying on HTML entities, which causes serious system malfunction. When you import events, it uses HTML entities instead of normal characters. They are arguing, that this is security issue, which is really lame excuse for serious bug.

    We do not write titles like these “I'm human & doesn't like HTML entities!”. Instead we write “I’m human & doesn’t like HTML entities!”

    So we except some sort of solution to this very serious issue.

    I’m 100% certain that this is a VERY SERIOUS BUG!

    For firs WordPress uses utf8 character set!!! Thats the fact! So stay on WordPress defaults!

    For second humans does not type HTML entities, its only for machines.

    For third “BB & DD Horses” <> “BB &amp; DD Horses” !

    For fourth, you are using function get_page_by_title to check if organiser exists already in the database. And get_page_by_title(“BB &amp; DD Horses”) does not return the same result width get_page_by_title(“BB & DD Horses”)!!. Instead, it says there is no organiser defined and creates new organiser, which causes system behave abnormal.

    It’s not hard to fix it, and 100% certain it has nothing do to with security!!! You have to write tiny fix in your code on customer side by using html_entity_decode function to convert characters into normal!

    So, just please fix the BUG!

Viewing 4 replies - 1 through 4 (of 4 total)
  • Moderator Jan Dembowski

    (@jdembowski)

    Forum Moderator and Brute Squad

    WordPress uses UTF-8 character set.

    Debatable but OK.

    When you import events, it uses HTML entities instead of normal characters. They are arguing, that this is security issue, which is really lame excuse for serious bug.

    How is that a serious bug?

    Thread Starter WP Apps

    (@wp-apps)

    Jan,
    thank you for response!

    How is that a serious bug?

    Here is an example:

    echo ‘”Me & You” == “Me &amp; You” is ‘ . json_encode((“Me & You” == “Me &amp; You”)); // “Me & You” == “Me & You” is false

    $post_1 = get_page_by_title(‘Me &amp; You’); // null;

    $post_2 = get_page_by_title(‘Me & You’); // WP_Post ;

    So, when The Event Calendar does the check get_page_by_title but title is using HTML entities, then it does not find the item from database.

    Simple!

    Barry

    (@barryhughes-1)

    Hi @wp-apps,

    Thanks for the feedback – if there’s a bug here, we are of course interested in resolving it.

    The developers of “The Events Calendar” does not appreciate WordPress standards

    I do take issue with the above statement, though.

    We take WordPress coding standards very seriously. There are certainly parts of our codebase written several years ago that don’t entirely meet the current standards (as is true of WordPress itself) and, as with any large project, there are undoubtedly a number of inconsistencies and oversights even in newer code — but that hardly means we are cavalier in our approach to these things.

    WordPress uses UTF-8 character set. The “The Events Calendar” developers are relaying on HTML entities, which causes serious system malfunction.

    OK – but would it be possible for you to distill the problem into a set of steps that will allow us to easily replicate and identify whichever part of the plugin this issue relates to?

    I suspect we’re talking about import functionality here, but I’m unsure if that is the built-in CSV import or some other source. If we could pin down a few specifics, that would be really helpful ??

    Hey there! This thread has been inactive for a while so we’re going to go ahead and mark it Resolved. Please feel free to open a new thread if any other questions come up and we’d be happy to help. ??

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Serious malfunction –’ is closed to new replies.