Install updates - Githubissues

gocodebox / lifterlms-helper

Update, install, and beta test LifterLMS and LifterLMS add-ons

0 stars 2 forks source link

Install updates #29

Closed thomasplevy closed 3 years ago

thomasplevy commented 3 years ago

Description

Fixes #28
Uses the proper constant that I missed when reviewing #26
Replaces usage of LLMS_Helper() with llms_helper() in the LLMS_Helper_Install class

How has this been tested?

New unit tests

Screenshots

Types of changes

Bug fix
This creates a scenario in which users on versions earlier that 3.0 will not have keys migrated if they delete the helper and then upgrade to LifterLMS 5.0 or later. In the end this would be a minor issue and we can instruct users to migrate their keys manually or install the helper plugin, migrate, and then delete.

Checklist:

[x] My code has been tested.
[x] My code passes all existing automated tests.
[x] My code follows the LifterLMS Coding & Documentation Standards.