Hilarious coding practise, lack of awareness of performance
-
This plugin works, we have it in use since a long time – but we have to patch the codebase over and over again with every update we do. Performance-wise, this plugin is a joke – they waste a lot of unnecessary SQL queries upon EVERY pageload. Never seen something similar – but they do their full initialization queries within the frontend and within every AJAX call.
We checked using Query Monitor: Upon every page refresh using F5, we see queries like these being performed – have you every seen something like this? This all belongs into a “once upon plugin activation init-routine“
CREATE TABLE IF NOT EXISTS wp_sib_model_users ( id int(20) NOT NULL AUTO_INCREMENT, email varchar(255), code varchar(100), listIDs longtext, redirectUrl varchar(255), info TEXT CHARACTER
SET utf8 COLLATE utf8_unicode_ci, frmid int(2), user_added_date DATETIME NOT NULL, PRIMARY KEY (id) );CREATE TABLE IF NOT EXISTS wp_sib_model_lang ( id int(20) NOT NULL AUTO_INCREMENT, frmID int(20) NOT NULL DEFAULT -1, pID int(20) NOT NULL DEFAULT -1, lang varchar(120), PRIMARY KEY (id) );
SHOW COLUMNS
FROM wp_sib_model_forms LIKE 'gCaptcha'SHOW COLUMNS
FROM wp_sib_model_forms LIKE 'termAccept';SHOW COLUMNS
FROM wp_sib_model_forms LIKE 'confirmID';SHOW COLUMNS
FROM wp_sib_model_forms LIKE 'requiredMsg';SHOW TABLES LIKE 'wp_sib_model_forms';
SHOW COLUMNS
FROM wp_sib_model_forms LIKE 'cCaptchaStyle'SHOW TABLES LIKE 'sib_model_forms'
Some of these queries run multiple times, their results are not even cached. They have several functions like these:
public static function forms_table_exists()
{
global $wpdb;
return $wpdb->get_var( "SHOW TABLES LIKE '" . self::TABLE_NAME . "'" ) == self::TABLE_NAME;
}Performance-wise, this is a complete joke. Luckily, not all devs work like this – just imagine having 20 plugins installed and every plugin spending 10-20 unnecessary SQL calls – you’d end up with 100 WASTED sql queries rather fast. No wonder that most WP pages are slow af.
If you can, pick a different mail provider instead of Brevo / SIB.
- You must be logged in to reply to this review.