72% pushvault-push-notifications

Code Review | Pushvault Push Notifications

WordPress plugin Pushvault Push Notifications scored 72% from 54 tests.

About plugin

  • Plugin page: pushvault-push-no...
  • Plugin version: 1.0
  • PHP compatiblity: 5.2.4+
  • PHP version: 7.4.16
  • WordPress compatibility: 4.2-4.9.9
  • WordPress version: 6.3.1
  • First release: Nov 21, 2018
  • Latest release: Dec 13, 2018
  • Number of updates: 17
  • Update frequency: every 1.7 days
  • Top authors: spirooxmedia (100%)

Code review

54 tests

User reviews

1 review

Install metrics

10+ active / 676 total downloads

Benchmarks

Plugin footprint 83% from 16 tests

Installer Passed 1 test

🔺 Critical test (weight: 50) | All plugins must install correctly, without throwing any errors, warnings, or notices
This plugin's installer ran successfully

Server metrics [RAM: ▲0.02MB] [CPU: ▼9.34ms] Passed 4 tests

Analyzing server-side resources used by Pushvault Push Notifications
No issues were detected with server-side resource usage
PageMemory (MB)CPU Time (ms)
Home /3.50 ▲0.0438.65 ▼0.31
Dashboard /wp-admin3.33 ▲0.0246.65 ▼21.85
Posts /wp-admin/edit.php3.38 ▲0.0246.21 ▼1.78
Add New Post /wp-admin/post-new.php5.91 ▲0.0296.77 ▼13.42
Media Library /wp-admin/upload.php3.25 ▲0.0236.05 ▲2.52
Webpush Notifications /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php3.2230.68

Server storage [IO: ▲0.08MB] [DB: ▲0.00MB] Passed 3 tests

How much does this plugin use your filesystem and database?
This plugin installed successfully
Filesystem: 22 new files
Database: no new tables, 6 new options
New WordPress options
theysaidso_admin_options
widget_theysaidso_widget
can_compress_scripts
widget_recent-posts
widget_recent-comments
db_upgraded

Browser metrics Passed 4 tests

A check of browser resources used by Pushvault Push Notifications
There were no issues detected in relation to browser resource usage
PageNodesMemory (MB)Script (ms)Layout (ms)
Home /2,938 ▲20313.62 ▼1.135.55 ▲3.8438.38 ▼3.53
Dashboard /wp-admin2,212 ▲245.90 ▲0.86103.15 ▼8.6840.50 ▲2.44
Posts /wp-admin/edit.php2,096 ▲42.17 ▲0.1834.61 ▼6.7933.86 ▼1.53
Add New Post /wp-admin/post-new.php1,548 ▲618.55 ▲0.20593.34 ▼26.6455.14 ▼7.87
Media Library /wp-admin/upload.php1,398 ▲74.24 ▼0.0095.83 ▼4.3343.58 ▼1.45
Webpush Notifications /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php8171.6724.3427.70

Uninstaller [IO: ▲0.00MB] [DB: ▲0.00MB] 75% from 4 tests

🔸 Tests weight: 35 | Verifying that this plugin uninstalls completely without leaving any traces
It is recommended to fix the following
  • Zombie WordPress options detected upon uninstall: 6 options
    • widget_recent-comments
    • widget_theysaidso_widget
    • can_compress_scripts
    • theysaidso_admin_options
    • widget_recent-posts
    • db_upgraded

Smoke tests 25% from 4 tests

Server-side errors 0% from 1 test

🔹 Test weight: 20 | A shallow check that no server-side errors were triggered
Almost there, just fix the following server-side errors
    • > GET request to /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php
    • > Notice in wp-content/plugins/pushvault-push-notifications/includes/pushvault_adminpage_frontend.php+35
    Undefined variable: error
    • > GET request to /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php
    • > Notice in wp-content/plugins/pushvault-push-notifications/includes/pushvault_adminpage_frontend.php+36
    Undefined index: pvlt-uid

SRP 50% from 2 tests

🔹 Tests weight: 20 | The single-responsibility principle applies for WordPress plugins as well - please make sure your PHP files perform no actions when accessed directly
The following issues need your attention
  • 1× PHP files perform the task of outputting text when accessed with GET requests:
    • > /wp-content/plugins/pushvault-push-notifications/pushvault.php

User-side errors 0% from 1 test

🔹 Test weight: 20 | Just a short smoke test targeting errors on the browser (console and network errors and warnings)
Please fix the following user-side errors
    • > GET request to /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php
    • > Console-api (warning) in unknown
    /wp-admin/load-scripts.php?c=0&load%5Bchunk_0%5D=jquery-core,jquery-migrate,utils&ver=6.3.1 1:28609 "jQuery.Deferred exception: Cannot read properties of undefined (reading 'model')" "TypeError: Cannot read properties of undefined (reading 'model')\n at HTMLDocument.\u003Canonymous> (/wp-admin/options-general.php?page=pushvault-push-notifications%2Fincludes%2Fpushvault_adminpage_frontend.php:170:36)\n at e (/wp-admin/load-scripts.php?c=0&load%5Bchunk_0%5D=jquery-core,jquery-migrate,utils&ver=6.3.1:2:26990)\n at t (/wp-admin/load-scripts.php?c=0&load%5Bchunk_0%5D=jquery-core,jquery-migrate,utils&ver=6.3.1:2:27292)" undefined
    • > GET request to /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php
    • > Javascript (severe) in unknown
    /wp-admin/load-scripts.php?c=0&load%5Bchunk_0%5D=jquery-core,jquery-migrate,utils&ver=6.3.1 1:28721 Uncaught TypeError: Cannot read properties of undefined (reading 'model')
    • > GET request to /wp-admin/options-general.php?page=pushvault-push-notifications/includes/pushvault_adminpage_frontend.php
    • > Network (severe)
    https://domains.squadhelp.com/lpd/name/pushvault.com - Failed to load resource: the server responded with a status of 403 ()

Optimizations

Plugin configuration 93% from 29 tests

readme.txt Passed 16 tests

The readme.txt file is an important file in your plugin as it is parsed by WordPress.org to prepare the public listing of your plugin
8 plugin tags: webmaster, pushvault, notification, push notification, push...

pushvault-push-notifications/pushvault.php 85% from 13 tests

"Pushvault Push Notifications" version 1.0's primary PHP file adds more information about the plugin and serves as the entry point for WordPress
Please take the time to fix the following:
  • Main file name: It is recommended to name the main PHP file as the plugin slug ("pushvault-push-notifications.php" instead of "pushvault.php")
  • Description: If Twitter did it, so should we! Keep the description under 140 characters (currently 159 characters long)

Code Analysis Passed 3 tests

File types Passed 1 test

🔸 Test weight: 35 | This is a short overview of programming languages used in this plugin, detecting executable files
Good job! No executable or dangerous file extensions detected813 lines of code in 15 files:
LanguageFilesBlank linesComment linesLines of code
PHP101529618
CSS244173
JavaScript20021
JSON1001

PHP code Passed 2 tests

Analyzing logical lines of code, cyclomatic complexity, and other code metrics
Everything seems fine, there were no complexity issues found
Cyclomatic complexity
Average complexity per logical line of code0.38
Average class complexity0.00
▷ Minimum class complexity0.00
▷ Maximum class complexity0.00
Average method complexity0.00
▷ Minimum method complexity0.00
▷ Maximum method complexity0.00
Code structure
Namespaces0
Interfaces0
Traits0
Classes0
▷ Abstract classes00.00%
▷ Concrete classes00.00%
▷ Final classes00.00%
Methods0
▷ Static methods00.00%
▷ Public methods00.00%
▷ Protected methods00.00%
▷ Private methods00.00%
Functions9
▷ Named functions9100.00%
▷ Anonymous functions00.00%
Constants0
▷ Global constants00.00%
▷ Class constants00.00%
▷ Public constants00.00%

Plugin size Passed 2 tests

Image compression Passed 2 tests

PNG files should be compressed to save space and minimize bandwidth usage
2 PNG files occupy 0.01MB with 0.00MB in potential savings
Potential savings
Compression of 2 random PNG files using pngquant
FileSize - originalSize - compressedSavings
resources/default_logo.png7.59KB2.88KB▼ 62.09%
resources/pushvault_thumb.png1.82KB1.34KB▼ 26.33%