OK, c'est ce que je craignais mais je n'ai pas trouvé de ticket qui
traitait ce problème.

Tu as raison pour le nom de l'objet/table qui a été mal nommé à
l'origine : je l'avais remarqué mais vu que ça fonctionnait je préférais
ne pas trop toucher :)

Des conseils pour renommer cet objet ?
Je renomme les fichiers (sing.), les classes (sing.), les tables
(plur.). Et touchant à la base de données, je dois aussi gérer la migration.
Il y a autre chose ?


Le 25/05/2014 11:35, Julien Dombre a écrit :
> Bonjour,
> 
> Non ce n'est pas un bug mais un choix pour ne plus gérer les adjectifs
> en fin de noms de tables car ils engendrent trop de problèmes.
> En effet comment savoir que c'est un adjectif et qu'il ne faut pas gérer
> le pluriel ?
> De plus, pour moi une table = un objet -> donc c'est cet élément qui est
> signifiant pas l'adjectif.
> Enfin en anglais l'adjectif se place avant le nom généralement.
> Typiquement connectionratesguaranteed (si je comprend bien sa
> signification) devrait être guaranteedconnectionsrates.
> 
> Cordialement,
> 
> Julien Dombre
> 
> 
> 
> 
> Le 25/05/2014 03:37, DUVERGIER Claude a écrit :
>> Bonjour,
>>
>> Je suis en train de mettre à jour un plugin (connections
>> <https://forge.indepnet.net/projects/connections>) pour qu'il fonctionne
>> en 0.84.x
>> Une de ses tables se nomme
>> "glpi_plugin_connections_connectionratesguaranteed" (gère un dropdown de
>> débits garantis).
>>
>> Lorsque j'affiche le formulaire d'ajout/modification
>> ("plugins/connections/front/connection.form.php") le dropdown n'apparait
>> pas j'obtiens des erreurs comme suit dans le fichier "sql-errors.log" :
>>
>>> 2014-05-25 02:57:51
>>> *** MySQL query error:
>>> ***
>>> SQL: SHOW COLUMNS FROM
>>> `glpi_plugin_connections_connectionratesguaranteeds`
>>> Error: Table
>>> 'glpi.glpi_plugin_connections_connectionratesguaranteeds' doesn't exist
>>> Backtrace :
>>> ...\GLPI-0.84.5\inc\dbmysql.class.php&nbsp;:392        DBmysql->query()
>>> ...\GLPI-0.84.5\inc\commondbtm.class.php&nbsp;:267       
>>> DBmysql->list_fields()
>>> ...\GLPI-0.84.5\inc\commondbtm.class.php&nbsp;:2427       
>>> CommonDBTM->getEmpty()
>>> ...\GLPI-0.84.5\ajax\dropdownValue.php&nbsp;:106       
>>> CommonDBTM->maybeTemplate()
>>> ...\GLPI-0.84.5\inc\ajax.class.php&nbsp;:612        include()
>>> ...\GLPI-0.84.5\inc\dropdown.class.php&nbsp;:206        Ajax::dropdown()
>>> ...\GLPI-0.84.5\inc\commondbtm.class.php&nbsp;:3541       
>>> Dropdown::show()
>>> ...\GLPI-0.84.5\plugins\connections\inc\connection.class.php&nbsp;:311      
>>>  
>>> CommonDBTM::dropdown()
>>> ...\GLPI-0.84.5\plugins\connections\front\connection.form.php&nbsp;:115     
>>>   
>>> PluginConnectionsConnection->showForm()
>>> .../GLPI-0.84.5/plugins/connections/front/connection.form.php
>> Vu le nom recherché "..._connectionratesguaranteeds" (notez le "s" en
>> trop à la fin) je me dit que ça provient du système de conversion
>> singulier/pluriel.
>> Jusqu'en 0.83.91 la méthode "getPlural()" de "inc/db.function.php"
>> gérait justement ce cas :
>>> $rules = array(//'singular' => 'plural'
>>>                    ....
>>>                    'ed$'         => 'ed',  // case table without
>>> plural (ex. imported)
>>>                    ....
>>>                    );
>> Mais ça a été supprimé par la révision 19396
>> <https://forge.indepnet.net/projects/glpi/repository/revisions/19396/diff/trunk/inc/db.function.php>,
>>
>> d'après le message de commit
>> (<https://forge.indepnet.net/projects/glpi/repository/revisions/19396>)
>> ça serait lié à #2593 (<https://forge.indepnet.net/issues/2593> mais je
>> penche plutôt pour un commit en erreur.
>>
>> Il ne doit pas y avoir beaucoup de tables se finissant par "...ed" mais
>> c'est le cas de ce plugin.
>>
>> Si s'avère qu'il s'agit bien d'un bug, est-ce quelqu'un pourrait ouvrir
>> une nouvelle demande sur la forge ? J'y posterait toutes ces
>> informations et le correctif (qui sera assez évident ^^).
>>
>> Merci
>> -- 
>> DUVERGIER Claude

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

Reply via email to