[#9522] - Additional module classes doesn't show - quick fix
- Closed
- 5 Apr 2016
- Medium
- Build: master
- # 9522
Steps to reproduce the issue
1) Install J3.5 in a PHP 5.3 environment
2) Install the plugin nnbootstrapparams (https://github.com/nn-medienagentur/plg_system_nnbootstrapparams)
3) Install a bootstrapped template like masterBootstrap (http://www.masterbootstrap.com)
4) Add a few modules to the feature position and play with the bootstrap params in each module.
Expected result
In J3.4-8 you will see the additional bootstrap classes being added to each module rendering.
Actual result
In J3.5 none of the additional boostrap classes are being shown.
Quick Fix (temp. hack)
In libraries > joomla > document > document.php
change line 999 from:
if (!class_exists($class))
to
if (class_exists($class))
Votes
Thank you for pointing that out. The quick (and dirty) fix solved the issue for me at the moment, however I would love to see a proper fix to solve the problem.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9522.
| Priority | Critical | ⇒ | Medium |
Rest priority according to docs https://docs.joomla.org/Priority
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9522.
Thank you, Brian.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9522.
Hrm. This does break BC.
NNBootstrapparams already updated to fix the issue: nn-medienagentur/plg_system_nnbootstrapparams#5
Thank you, impleri! Problem solved :)
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-05 07:16:30 |
| Closed_By | ⇒ | jerndorff |
That's not a valid fix. Joomla 3.5 restructured the JDocument internal classes and the renderer loading function here gives preference to the new convention over the old. Your change forces it to always use the old convention which would defeat the point of the change to begin with.