Update edd-software-license-manager.php#8
Conversation
This is part 1 of 3 changes required to add License Term as a 'global' setting in edd-software-license-manager. Apparently I did the wrong the first time because I can't find the first change I made to this file. This is my first GitHub change...so I might probably have done something incorrectly. Please forgive my ignorance.
It might be argued that there could and possibly should be more filters than this one. It turns out that the other settings are acceptable to me when the prior pull requests are also applied. The problem I'm trying to solve is the ability to associate a specific product to a specific license key. The mechanism as it stands does not make this process easy. However, with the filter in place, the user of the edd_slm plugin can specify the transaction ID as needed to connect the license to the product more closely.
|
I fear I'm not an expert in github still. Please let me know what I need to do to help get these changes into an updated plugin. |
|
Hey @btwatts, it's fine so far, thanks for your help! I'll take a look into as soon as I find time and update the plugin |
|
I did several pulls that maybe could be only one pull. Thanks for taking a look.
|
|
I discovered a problem in my further testing that I believe is caused by my test for the existence of It appears that that early exit is causing the Extension Settings to break. It is also possible that it is causing the entire plugin to exit in some cases that should not. Please Verify Receipt |
2 participants
This is part 1 of 3 changes required to add License Term as a 'global' setting in edd-software-license-manager.
Apparently I did the wrong the first time because I can't find the first change I made to this file.
This is my first GitHub change...so I might probably have done something incorrectly.
Please forgive my ignorance.