two little bugs fixed in PluginManager
Posted: Wed Mar 11, 2015 4:09 pm
Hi everyone,
here is a little bug fix inside the model of the PluginManager.
There was a little inconsistency regarding the information collected from the learning_plugins table: the field "category" fetched in the SQL query string was ignored during the gather process inside getPluginsInfo, causing the returned array to be inconsistent (the active slot was filled with the description field, for example).
Moreover I also expose the value of the category field in the view of the page (it could be useful when you will have a lot - hope so! - of plugins available).
Another thing I notice is that there is a strange behavoiur regarding the field "priority" which you can set to 0 or 1 from the configurarion panel, but, as a matter of fact, the value of this field shouldn't be user definable...
Maybe here the Forma dev team could shed a light on this issue.
Please find my patch in the zip file in attachment. Just unpack the zip and substitute the original file
- [root_forma]/
---->appCore/
------>models
-------->PluginAdm.php (You can easily find the changes inside the code looking for my name inside the code)
There is another little bug I notice in 1.3 forma release: the file
[root_forma]/plugins/ConferenceTeleSkill/ConferenceTeleskillPlugin.php
must be renamed as
[root_forma]/plugins/ConferenceTeleSkill/ConferenceTeleSkillPlugin.php
otherwise you can happen to see a 500 internal server error (Fatal error caused by the fail in opening that file).
here is a little bug fix inside the model of the PluginManager.
There was a little inconsistency regarding the information collected from the learning_plugins table: the field "category" fetched in the SQL query string was ignored during the gather process inside getPluginsInfo, causing the returned array to be inconsistent (the active slot was filled with the description field, for example).
Moreover I also expose the value of the category field in the view of the page (it could be useful when you will have a lot - hope so! - of plugins available).
Another thing I notice is that there is a strange behavoiur regarding the field "priority" which you can set to 0 or 1 from the configurarion panel, but, as a matter of fact, the value of this field shouldn't be user definable...
Maybe here the Forma dev team could shed a light on this issue.
Please find my patch in the zip file in attachment. Just unpack the zip and substitute the original file
- [root_forma]/
---->appCore/
------>models
-------->PluginAdm.php (You can easily find the changes inside the code looking for my name inside the code)
There is another little bug I notice in 1.3 forma release: the file
[root_forma]/plugins/ConferenceTeleSkill/ConferenceTeleskillPlugin.php
must be renamed as
[root_forma]/plugins/ConferenceTeleSkill/ConferenceTeleSkillPlugin.php
otherwise you can happen to see a 500 internal server error (Fatal error caused by the fail in opening that file).