Opened 9 years ago
Closed 9 years ago
#40285 closed defect (bug) (invalid)
Themes in the Theme Directory should be required to use TGM-Plugin-Activation version of at least 2.6.1
| Reported by: | beeneeb | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Themes | Version: | 4.7.3 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
The theme "Emmet Lite" located in the Wordpress Theme Directory uses version 2.5.2 of TGM-Plugin-Activation. Not only is it a security issue, but also causes fatal errors with other plugins that use the latest version of TGM of 2.6.1.
https://wordpress.org/themes/emmet-lite/
To reproduce, install the Emmet Lite theme & then install this plugin:
https://wordpress.org/plugins/our-team-enhanced/
Expected:
No fatal errors
Actual:
Fatal error: Cannot redeclare tgmpa() (previously declared
Change History (3)
#2
@
9 years ago
Hi @beeneeb!
What you can also do is comment on the theme's latest trac ticket and let them know about this issue or even posting on their support forum: https://wordpress.org/support/theme/emmet-lite
#3
@
9 years ago
- Milestone Awaiting Review
- Resolution → invalid
- Status new → closed
This should be reported to the theme author directly - @MotoPress
I don't see any threads in the themes forum though related to it.
The code in question that's faulty is https://themes.trac.wordpress.org/browser/emmet-lite/1.4.1/classes/theme/class-tgm-plugin-activation.php?marks=1917-1927#L1900
Marking as invalid as this isn't a core WordPress problem.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Hi @beeneeb,
Just to clarify: this is not an issue with the TGMPA version being used, there is no conflict between TGMPA 2.4.1+ and higher versions.
This is a "typo" - or rather, search and replace error - made by the theme author who has manually adjusted the code in the TGMPA class.
Around line 1920, you will find a line that should read:
if ( ! function_exists( 'tgmpa' ) ) {but will have been changed to read:
if ( ! function_exists( 'theme-slug' ) ) {As they incorrectly changed it, the actual function which is wrapped within that if statement is being redefined causing that error.
Theme authors are advised to download a fresh copy of TGMPA whenever they release and/or upgrade a theme using the Custom TGMPA Generator. The Custom TGMPA Generator will serve them with a customized download of TGMPA which will automagically have done the search & replace for them in the correct way which prevents this issue.