88% accessally-lms-migration-from-zippy-courses

Code Review | AccessAlly™ LMS Migration from Zippy Courses®

WordPress plugin AccessAlly™ LMS Migration from Zippy Courses® scored 88% from 54 tests.

About plugin

  • Plugin page: accessally-lms-mi...
  • Plugin version: 1.0.1
  • PHP compatiblity: 5.6+
  • PHP version: 7.4.16
  • WordPress compatibility: 4.7.0-5.2.3
  • WordPress version: 5.9.2
  • First release: May 20, 2020
  • Latest release: May 3, 2021
  • Number of updates: 6
  • Update frequency: every 58.0 days
  • Top authors: rli (66.67%)accessally (50%)

Code review

54 tests

User reviews

1 review

Install metrics

10+ active / 454 total downloads


Plugin footprint Passed 16 tests

Installer Passed 1 test

🔺 Critical test (weight: 50) | Checking the installer triggered no errors
The plugin installed successfully, without throwing any errors or notices

Server metrics [RAM: ▲0.07MB] [CPU: ▼299.13ms] Passed 4 tests

An overview of server-side resources used by AccessAlly™ LMS Migration from Zippy Courses®
Server-side resource usage in normal parameters
PageMemory (MB)CPU Time (ms)
Home /3.68 ▲0.1654.88 ▲2.71
Dashboard /wp-admin3.51 ▲0.1033.88 ▼0.58
Posts /wp-admin/edit.php3.74 ▲0.0937.54 ▲0.33
Add New Post /wp-admin/post-new.php6.97 ▲0.0092.67 ▼1,197.67
Media Library /wp-admin/upload.php3.39 ▲0.0924.14 ▲1.39

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

How much does this plugin use your filesystem and database?
No storage issues were detected
Filesystem: 14 new files
Database: no new tables, no new options

Browser metrics Passed 4 tests

AccessAlly™ LMS Migration from Zippy Courses®: an overview of browser usage
This plugin has a minimal impact on browser resources
PageNodesMemory (MB)Script (ms)Layout (ms)
Home /3,782 ▲6015.80 ▼0.205.72 ▼2.022.17 ▼0.42
Dashboard /wp-admin2,911 ▲506.00 ▼0.17117.94 ▼18.80177.67 ▲14.92
Posts /wp-admin/edit.php2,690 ▲23.13 ▼0.1063.60 ▲0.34141.39 ▼1.30
Add New Post /wp-admin/post-new.php1,681 ▲2018.95 ▲1.19388.19 ▼22.07202.42 ▲47.10
Media Library /wp-admin/upload.php1,715 ▲175.55 ▲0.02123.32 ▼18.82207.76 ▲21.40

Uninstaller [IO: ▲0.00MB] [DB: ▲0.00MB] Passed 4 tests

🔸 Tests weight: 35 | All plugins must uninstall correctly, removing their source code and extra database tables they might have created
Uninstall script ran successfully

Smoke tests 50% from 4 tests

Server-side errors Passed 1 test

🔹 Test weight: 20 | A shallow check that no server-side errors were triggered
Even though no errors were found, this is by no means an exhaustive test

SRP 0% from 2 tests

🔹 Tests weight: 20 | A shallow check of the single-responsibility principle; PHP files should perform no action - including output of placeholder text - and trigger no errors when accessed directly
Almost there! Just fix the following items
  • 3× PHP files output non-empty strings when accessed directly via GET requests:
    • > /wp-content/plugins/accessally-lms-migration-from-zippy-courses/backend/settings-template.php
    • > /wp-content/plugins/accessally-lms-migration-from-zippy-courses/backend/existing-template.php
    • > /wp-content/plugins/accessally-lms-migration-from-zippy-courses/backend/convert-template.php
  • 2× PHP files trigger server errors when accessed directly:
    • > PHP Fatal error
      Uncaught Error: Call to undefined function plugin_dir_url() in wp-content/plugins/accessally-lms-migration-from-zippy-courses/accessally-lms-migrate-zippy-courses.php:27
    • > PHP Fatal error
      Uncaught Error: Call to undefined function _e() in wp-content/plugins/accessally-lms-migration-from-zippy-courses/backend/settings-display.php:2

User-side errors Passed 1 test

🔹 Test weight: 20 | This is just a short smoke test looking for browser issues
Everything seems fine, but this is not an exhaustive test


Plugin configuration 93% from 29 tests

readme.txt Passed 16 tests

The readme.txt file describes your plugin functionality and requirements and it is parsed to prepare the your plugin's listing
10 plugin tags: import lms, export zippy courses, lms migration, migrate lms, access ally...

accessally-lms-migration-from-zippy-courses/accessally-lms-migrate-zippy-courses.php 85% from 13 tests

The primary PHP file in "AccessAlly™ LMS Migration from Zippy Courses®" version 1.0.1 is used by WordPress to initiate all plugin functionality
It is important to fix the following:
  • Description: Keep the plugin description shorter than 140 characters (currently 197 characters long)
  • Main file name: Even though not officially enforced, the main plugin file should be the same as the plugin slug ("accessally-lms-migration-from-zippy-courses.php" instead of "accessally-lms-migrate-zippy-courses.php")

Code Analysis Passed 3 tests

File types Passed 1 test

🔸 Test weight: 35 | A short review of files and their extensions; it is not recommended to include executable files
No dangerous file extensions were detected864 lines of code in 7 files:
LanguageFilesBlank linesComment linesLines of code

PHP code Passed 2 tests

A short review of cyclomatic complexity and code structure
This plugin has no cyclomatic complexity issues
Cyclomatic complexity
Average complexity per logical line of code0.38
Average class complexity83.00
▷ Minimum class complexity83.00
▷ Maximum class complexity83.00
Average method complexity4.57
▷ Minimum method complexity1.00
▷ Maximum method complexity15.00
Code structure
▷ Abstract classes00.00%
▷ Concrete classes1100.00%
▷ Final classes00.00%
▷ Static methods23100.00%
▷ Public methods1147.83%
▷ Protected methods00.00%
▷ Private methods1252.17%
▷ Named functions00.00%
▷ Anonymous functions00.00%
▷ Global constants00.00%
▷ Class constants6100.00%
▷ Public constants6100.00%

Plugin size Passed 2 tests

Image compression Passed 2 tests

It is recommended to compress PNG files in your plugin to minimize bandwidth usage
5 PNG files occupy 0.02MB with 0.01MB in potential savings
Potential savings
Compression of 5 random PNG files using pngquant
FileSize - originalSize - compressedSavings
backend/completed.png1.43KB0.69KB▼ 51.40%
backend/pencil.png15.01KB0.21KB▼ 98.63%