77% accessally-lms-migration-from-wp-courseware

Code Review | AccessAlly™ LMS Migration from WP Courseware®

WordPress plugin AccessAlly™ LMS Migration from WP Courseware® scored 77% 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.5.3
  • WordPress version: 6.3.1
  • First release: Nov 21, 2020
  • Latest release: Nov 22, 2020
  • Number of updates: 3
  • Update frequency: every 0.7 days
  • Top authors: accessally (100%)

Code review

54 tests

User reviews

1 review

Install metrics

10+ active / 561 total downloads

Benchmarks

Plugin footprint 83% from 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.01MB] [CPU: ▼0.85ms] Passed 4 tests

An overview of server-side resources used by AccessAlly™ LMS Migration from WP Courseware®
This plugin has minimal impact on server resources
PageMemory (MB)CPU Time (ms)
Home /3.47 ▲0.0135.28 ▼5.41
Dashboard /wp-admin3.32 ▲0.0146.70 ▲0.88
Posts /wp-admin/edit.php3.43 ▲0.0747.97 ▲2.62
Add New Post /wp-admin/post-new.php5.90 ▲0.0183.69 ▼1.48
Media Library /wp-admin/upload.php3.24 ▲0.0136.91 ▲3.40

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

Filesystem and database footprint
There were no storage issued detected upon installing this plugin
Filesystem: 14 new files
Database: no new tables, 6 new options
New WordPress options
widget_recent-comments
db_upgraded
theysaidso_admin_options
can_compress_scripts
widget_recent-posts
widget_theysaidso_widget

Browser metrics Passed 4 tests

A check of browser resources used by AccessAlly™ LMS Migration from WP Courseware®
This plugin renders optimally with no browser resource issues detected
PageNodesMemory (MB)Script (ms)Layout (ms)
Home /2,840 ▲8313.26 ▼1.031.74 ▼0.4341.95 ▼4.12
Dashboard /wp-admin2,224 ▲334.99 ▲0.09102.98 ▼9.0468.46 ▲25.32
Posts /wp-admin/edit.php2,104 ▲151.97 ▼0.0239.27 ▲5.5735.03 ▼0.10
Add New Post /wp-admin/post-new.php1,680 ▲16621.78 ▲4.10605.77 ▼86.5155.74 ▲0.51
Media Library /wp-admin/upload.php1,400 ▲94.14 ▼0.07112.90 ▲15.2143.56 ▲1.45

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

🔸 Tests weight: 35 | All plugins must uninstall correctly, removing their source code and extra database tables they might have created
The following items require your attention
  • The uninstall procedure has failed, leaving 6 options in the database
    • widget_theysaidso_widget
    • db_upgraded
    • can_compress_scripts
    • widget_recent-posts
    • theysaidso_admin_options
    • widget_recent-comments

Smoke tests 50% from 4 tests

Server-side errors Passed 1 test

🔹 Test weight: 20 | This is a short smoke test looking for server-side errors
The smoke test was a success, however most plugin functionality was not tested

SRP 0% from 2 tests

🔹 Tests weight: 20 | SRP (Single-Responsibility Principle) - PHP files must act as libraries and never output text or perform any action when accessed directly in a browser
Please fix the following items
  • 3× PHP files perform the action of outputting non-empty strings when accessed directly:
    • > /wp-content/plugins/accessally-lms-migration-from-wp-courseware/backend/settings-template.php
    • > /wp-content/plugins/accessally-lms-migration-from-wp-courseware/backend/convert-template.php
    • > /wp-content/plugins/accessally-lms-migration-from-wp-courseware/backend/existing-template.php
  • 2× PHP files trigger server errors when accessed directly:
    • > PHP Fatal error
      Uncaught Error: Call to undefined function _e() in wp-content/plugins/accessally-lms-migration-from-wp-courseware/backend/settings-display.php:2
    • > PHP Fatal error
      Uncaught Error: Call to undefined function plugin_dir_url() in wp-content/plugins/accessally-lms-migration-from-wp-courseware/accessally-lms-migrate-wp-courseware.php:25

User-side errors Passed 1 test

🔹 Test weight: 20 | This is a smoke test targeting browser errors/issues
There were no browser issues found

Optimizations

Plugin configuration 83% from 29 tests

readme.txt 81% from 16 tests

You should put a lot of thought into formatting readme.txt as it is used by WordPress.org to prepare the public listing of your plugin
These attributes need your attention:
  • Screenshots: These screenshots do not have images: #1 (See existing WP Courseware courses), #2 (Decide what type of conversion you want to perform.), #3 (Simple LMS migration with the click of the convert button.), #4 (Once the conversion is complete, you can revert or click to edit the course directly in AccessAlly's course wizard.), #5 (Just follow the steps through the AccessAlly Course Wizard to finish setting up your course, and make any updates to your course organization.), #6 (You'll be able to edit and save your new posts and make changes from there.)
  • Screenshots: Add descriptions for screenshots #0, #0, #0, #0, #0, #0 in accessally-lms-migration-from-wp-courseware/assets to your readme.txt
  • Tags: Too many tags (11 tag instead of maximum 10); only the first 5 tags are used in your directory listing
You can look at the official readme.txt

accessally-lms-migration-from-wp-courseware/accessally-lms-migrate-wp-courseware.php 85% from 13 tests

The main PHP file in "AccessAlly™ LMS Migration from WP Courseware®" ver. 1.0.1 adds more information about the plugin and also serves as the entry point for this plugin
Please take the time to fix the following:
  • Main file name: It is recommended to name the main PHP file as the plugin slug ("accessally-lms-migration-from-wp-courseware.php" instead of "accessally-lms-migrate-wp-courseware.php")
  • Description: If Twitter did it, so should we! Keep the description under 140 characters (currently 201 characters long)

Code Analysis Passed 3 tests

File types Passed 1 test

🔸 Test weight: 35 | Executable files are not allowed as they can serve as attack vectors
Everything looks great! No dangerous files found in this plugin844 lines of code in 7 files:
LanguageFilesBlank linesComment linesLines of code
PHP55029465
CSS102249
JavaScript157130

PHP code Passed 2 tests

Analyzing logical lines of code, cyclomatic complexity, and other code metrics
Although this was not an exhaustive test, there were no cyclomatic complexity issues detected
Cyclomatic complexity
Average complexity per logical line of code0.35
Average class complexity73.00
▷ Minimum class complexity73.00
▷ Maximum class complexity73.00
Average method complexity4.27
▷ Minimum method complexity1.00
▷ Maximum method complexity13.00
Code structure
Namespaces0
Interfaces0
Traits0
Classes1
▷ Abstract classes00.00%
▷ Concrete classes1100.00%
▷ Final classes00.00%
Methods22
▷ Static methods22100.00%
▷ Public methods1150.00%
▷ Protected methods00.00%
▷ Private methods1150.00%
Functions0
▷ Named functions00.00%
▷ Anonymous functions00.00%
Constants3
▷ Global constants00.00%
▷ Class constants3100.00%
▷ Public constants3100.00%

Plugin size Passed 2 tests

Image compression Passed 2 tests

Often times overlooked, PNG files can occupy unnecessary space in your plugin
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/revert.png0.38KB0.48KB0.00%
backend/completed.png1.43KB0.69KB▼ 51.40%
backend/icon.png0.70KB0.75KB0.00%
backend/pencil.png15.01KB0.21KB▼ 98.63%
backend/error.png2.42KB2.56KB0.00%