Split: WordPress Database – Unknown Column

Home Forums AdRotate for WordPress Bug Reports Split: WordPress Database – Unknown Column

  • #154072

    athomasRaka
    Badges:

    Even if the database is updated, the code is invalid for any MySQL database using specific strict modes which are now defaults for MySQL v8.

    The issue is when you are using backticks for strings and they should be using single quotes.
    Example:
    $wpdb->query("UPDATE {$wpdb->prefix}adrotate SET type = limit WHERE id = {$banner->id};");

    I submitted a lengthy bug report about this in March and now I do not see that report. Is it possible to find that report or was it deleted?

    #154074

    Arnan de Gans
    Badges:

    AdRotate doesn’t use backticks for strings. So that’s no problem.
    I didn’t see a bug report from you. However, you can post it in the bug reports forum or send it via email.

    https://ajdg.solutions/forums/forum/adrotate-for-wordpress/bug-reports/
    https://ajdg.solutions/contact/

    #154076

    athomasRaka
    Badges:

    The code snippet I pasted is directly from your plugin, adrotate-functions.php line 206. The part that is type = limit uses backticks for both of those. The limit part should be enclosed in single quotes. There are multiple instances of this in that file as well.
    I had to manually update the code to fix this but I can undo it to prove to you that the issue exists, just let me know.

    #154078

    Arnan de Gans
    Badges:

    Hmm, I see what you mean. You found a bug 😉
    Your suggestion that this is a plugin wide problem is still not accurate though as that is not the norm in AdRotate (Pro).

    #154085

    Arnan de Gans
    Badges:

    Split and moved to bug reports.

    #154091

    athomasRaka
    Badges:

    I appreciate the acknowledgement on the bug. I must have failed at submitting the report in March before… who knows.

    As for the other instances, I believe there are 5 total:

    • limit – line 206
    • limit – line 227
    • limit – line 275
    • limit – line 1071
    • budget – line 281

    Thanks!

    #154094

    Arnan de Gans
    Badges:

    I’ll have a look. Thanks!

    #154101

    Arnan de Gans
    Badges:

    I found one more actually 🙄

    But it’ll be fixed for the next version.
    Thanks Again.

    #154189

    Arnan de Gans
    Badges:
Viewing 9 posts - 1 through 9 (of 9 total)

The topic ‘Split: WordPress Database – Unknown Column’ is closed to new replies.

You may be interested in