Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not working with complianz-gdpr #123

Open
CommanderRedYT opened this issue Jun 18, 2024 · 0 comments
Open

Not working with complianz-gdpr #123

CommanderRedYT opened this issue Jun 18, 2024 · 0 comments

Comments

@CommanderRedYT
Copy link

CommanderRedYT commented Jun 18, 2024

This is the relevant log output.

Two problems in my mind:

  1. I do not know why the default is not escaped.
  2. It is also using SHOW COLUMNS, which is why I created a MR (Implement "SHOW COLUMNS" rewrite #122) for it because as far as I saw only SHOW FULL COLUMNS is supported right now.
wordpress-wordpress-1  | NOTICE: PHP message: WordPress database error ERROR:  syntax error at or near "default"
wordpress-wordpress-1  | LINE 5:              default int NOT NULL,
wordpress-wordpress-1  |                      ^ for query CREATE TABLE wp_cmplz_cookiebanners (
wordpress-wordpress-1  |              `ID` int(11) NOT NULL AUTO_INCREMENT,
wordpress-wordpress-1  |              `title` text NOT NULL,
wordpress-wordpress-1  |              `banner_version` int(11) NOT NULL,
wordpress-wordpress-1  |              `default` int(11) NOT NULL,
wordpress-wordpress-1  |             `position` text NOT NULL,
wordpress-wordpress-1  |             `checkbox_style` text NOT NULL,
wordpress-wordpress-1  |             `use_logo` text NOT NULL,
wordpress-wordpress-1  |             `logo_attachment_id` text NOT NULL,
wordpress-wordpress-1  |                        `close_button` text NOT NULL,
wordpress-wordpress-1  |             `revoke` text NOT NULL,
wordpress-wordpress-1  |             `manage_consent_options` text NOT NULL,
wordpress-wordpress-1  |             `header` text NOT NULL,
wordpress-wordpress-1  |             `dismiss` text NOT NULL,
wordpress-wordpress-1  |             `save_preferences` text NOT NULL,
wordpress-wordpress-1  |             `view_preferences` text NOT NULL,
wordpress-wordpress-1  |             `category_functional` text NOT NULL,
wordpress-wordpress-1  |             `category_all` text NOT NULL,
wordpress-wordpress-1  |             `category_stats` text NOT NULL,
wordpress-wordpress-1  |             `category_prefs` text NOT NULL,
wordpress-wordpress-1  |             `accept` text NOT NULL,
wordpress-wordpress-1  |             `message_optin` text NOT NULL,
wordpress-wordpress-1  |             `use_categories` text NOT NULL,
wordpress-wordpress-1  |             `disable_cookiebanner` int(11) NOT NULL,
wordpress-wordpress-1  |             `banner_width` int(11) NOT NULL,
wordpress-wordpress-1  |             `soft_cookiewall` int(11) NOT NULL,
wordpress-wordpress-1  |             `dismiss_on_scroll` int(11) NOT NULL,
wordpress-wordpress-1  |             `dismiss_on_timeout` int(11) NOT NULL,
wordpress-wordpress-1  |             `dismiss_timeout` text NOT NULL,
wordpress-wordpress-1  |             `accept_informational` text NOT NULL,
wordpress-wordpress-1  |             `message_optout` text NOT NULL,
wordpress-wordpress-1  |             `use_custom_cookie_css` text NOT NULL,
wordpress-wordpress-1  |             `custom_css` text NOT NULL,
wordpress-wordpress-1  |             `statistics` text NOT NULL,
wordpress-wordpress-1  |             `functional_text` text NOT NULL,
wordpress-wordpress-1  |             `statistics_text` text NOT NULL,
wordpress-wordpress-1  |             `statistics_text_anonymous` text NOT NULL,
wordpress-wordpress-1  |             `preferences_text` text NOT NULL,
wordpress-wordpress-1  |             `marketing_text` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_background` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_text` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_toggles` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_border_radius` text NOT NULL,
wordpress-wordpress-1  |             `border_width` text NOT NULL,
wordpress-wordpress-1  |             `font_size` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_button_accept` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_button_deny` text NOT NULL,
wordpress-wordpress-1  |             `colorpalette_button_settings` text NOT NULL,
wordpress-wordpress-1  |             `buttons_border_radius` text NOT NULL,
wordpress-wordpress-1  |             `animation` text NOT NULL,
wordpress-wordpress-1  |             `use_box_shadow` int(11) NOT NULL,
wordpress-wordpress-1  |             `header_footer_shadow` int(11) NOT NULL,
wordpress-wordpress-1  |             `hide_preview` int(11) NOT NULL,
wordpress-wordpress-1  |             `disable_width_correction` int(11) NOT NULL,
wordpress-wordpress-1  |             `legal_documents` int(11) NOT NULL,
wordpress-wordpress-1  |               PRIMARY KEY  (ID)
wordpress-wordpress-1  |             ) DEFAULT CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_520_ci made by require_once('wp-load.php'), require_once('wp-config.php'), require_once('wp-settings.php'), do_action('init'), WP_Hook->do_action, WP_Hook->apply_filters, cmplz_check_upgrade, do_action('cmplz_install_tables'), WP_Hook->do_action, WP_Hook->apply_filters, cmplz_install_cookiebanner_table, dbDelta
wordpress-wordpress-1  | NOTICE: PHP message: PHP Fatal error:  Uncaught Exception: Invalid or unsupported SQL statement. in /var/www/html/pg4wp/driver_pgsql_rewrite.php:26
wordpress-wordpress-1  | Stack trace:
wordpress-wordpress-1  | #0 /var/www/html/pg4wp/driver_pgsql_rewrite.php(40): createSQLRewriter('SHOW COLUMNS FR...')
wordpress-wordpress-1  | #1 /var/www/html/pg4wp/driver_pgsql.php(523): pg4wp_rewrite('SHOW COLUMNS FR...')
wordpress-wordpress-1  | #2 /var/www/html/pg4wp/core.php(34) : eval()'d code(2349): wpsqli_query(Object(PgSql\Connection), 'SHOW COLUMNS FR...')
wordpress-wordpress-1  | #3 /var/www/html/pg4wp/core.php(34) : eval()'d code(2263): wpdb2->_do_query('SHOW COLUMNS FR...')
wordpress-wordpress-1  | #4 /var/www/html/pg4wp/core.php(34) : eval()'d code(3146): wpdb2->query('SHOW COLUMNS FR...')
wordpress-wordpress-1  | #5 /var/www/html/wp-content/plugins/complianz-gdpr/cookiebanner/class-cookiebanner.php(88): wpdb2->get_results('SHOW COLUMNS FR...')
wordpress-wordpress-1  | #6 /var/www/html/wp-includes/class-wp-hook.php(324): cmplz_install_cookiebanner_table('')
wordpress-wordpress-1  | #7 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array)
wordpress-wordpress-1  | #8 /var/www/html/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
wordpress-wordpress-1  | #9 /var/www/html/wp-content/plugins/complianz-gdpr/upgrade.php(27): do_action('cmplz_install_t...')
wordpress-wordpress-1  | #10 /var/www/html/wp-includes/class-wp-hook.php(324): cmplz_check_upgrade('')
wordpress-wordpress-1  | #11 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array)
wordpress-wordpress-1  | #12 /var/www/html/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
wordpress-wordpress-1  | #13 /var/www/html/wp-settings.php(695): do_action('init')
wordpress-wordpress-1  | #14 /var/www/html/wp-config.php(133): require_once('/var/www/html/w...')
wordpress-wordpress-1  | #15 /var/www/html/wp-load.php(50): require_once('/var/www/html/w...')
wordpress-wordpress-1  | #16 /var/www/html/wp-admin/admin.php(34): require_once('/var/www/html/w...')
wordpress-wordpress-1  | #17 {main}
wordpress-wordpress-1  |   thrown in /var/www/html/pg4wp/driver_pgsql_rewrite.php on line 26

Originally posted by @CommanderRedYT in #77 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants