Hi,

I forgot the use of canCreate method for the ability to create a drop down element. That is the better solution : forget the dropdown patch I propose.

Regards
    Damien Touraine
On 07/21/11 12:15, MoYo wrote:
Hi,

Ok for all patchs except the last one on dropdown class : can_create must be manage using canCreate function of the itemtype class.

Patch for getPlural / getSingular is very intresting. I did not think such a solution.

Will applied them tommorrow.

Regards

Julien



Le 21/07/2011 12:10, Damien Touraine a écrit :
Hello,
I have send major updates for the network addresses management.
However, I have several few minor suggestions that can enhance the current SVN release.

As attached, you will find the "svn diff" patch of some files :
* commontreedropdown.class.php : I suggest to use a specific specialisation regarding the current class to define the completename of the CommonTreeDropDown class. That may be usefull when the class have to show another complete name than the default one (for instance, a complete internet domain name should be represented with dot notations). * db.function.php : the management of getPlural and getSingular may be surprising. Actually, with the previous solution, we had to worry of the last rules that add or remove the final 's'. (we had to append a specific 's' in special cases). In the version I suggest, we count the number of replacement that have been done. If there is at least one replacement, then it means the current rule is the good one. So, we don't have to continue. Thus, we don't have to worry about the following cases. * dropdown class : I think there can be drop downs that don't allow users to append elements. So I propose to add a "can_create" parameter (same way than the "comments" parameter).

KR
    Damien Touraine


_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


--
--------------------------------------------------------------------
Damien TOURAINE - Ingénieur de Recherche CNRS, LIMSI-CNRS
Groupe de RV&A "VENISE", (http://www.limsi.fr/venise/)
Bat. 508, Universite Paris-Sud 91403 Orsay cedex - +33 1 69 85 81 64
--------------------------------------------------------------------

_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Reply via email to