Bug #4620

Child entity for template

Added by yllen over 7 years ago. Updated over 7 years ago.

Status:ClosedStart date:10/25/2013
Priority:NormalDue date:
Assignee:yllen% Done:

100%

Category:Other
Target version:0.84.3

Associated revisions

Revision 22103
Added by yllen over 7 years ago

child entities for template - fixed #4620

Revision 22109
Added by yllen over 7 years ago

[0.84] child entity for contract template - see #4620

Revision 22111
Added by yllen over 7 years ago

[0.83] child entity for contract template - see #4620

Revision 22113
Added by yllen over 7 years ago

[0.84] child entity for contract template - see #4620

Revision 22114
Added by yllen over 7 years ago

child entity for contract template - fixed #4620

Revision 22115
Added by yllen over 7 years ago

[0.84] revert child entity for template - see #4620

Revision 22116
Added by yllen over 7 years ago

[0.83] revert child entity for template - see #4620

Revision 22117
Added by yllen over 7 years ago

revert child entity for template - see #4620

Revision 22118
Added by yllen over 7 years ago

[0.84] createcontract for parent template - see #4620

Revision 22119
Added by yllen over 7 years ago

[0.84] better fix - see #4620

Revision 22128
Added by moyo over 7 years ago

[0.84] Child entity for template fixed #4620

Revision 22129
Added by moyo over 7 years ago

Child entity for template see #4620

History

#2 Updated by yllen over 7 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r22103.

#3 Updated by moyo over 7 years ago

  • Status changed from Resolved to Assigned
  • % Done changed from 100 to 50

Je ne comprend pas vraiment la correction.
Pour les éléments qui peuvent être récursifs pourquoi bloquer la gestion de templates récursifs ?

C'est à l'ajout d'un élément basé sur un template qu'il y a un soucis je pense. Il faut dans ce cas définir la bonne entité si on n'a pas accès à l'entité du template.

#4 Updated by moyo over 7 years ago

Sur n'importe quel élément qui peut etre récursif la modif proposée provoque :
PHP Notice: Undefined property: Document::$input in /home/dombre/httpd/glpi084/inc/commondbtm.class.php at line 2074

#5 Updated by yllen over 7 years ago

Non, le problème vient lors de la création du template, il est proposé par défaut Sous entité Oui/Non

#6 Updated by yllen over 7 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 50 to 100

Applied in changeset r22114.

#7 Updated by moyo over 7 years ago

  • Status changed from Resolved to Assigned
  • % Done changed from 100 to 50

Désolé mais toujours pas d'accord avec la solution proposée...

Dans une fonction générique ajouté un == 'Contract' c'est juste pas possible (NetworkEquipment a aussi des templats et peut être récursif par exemple...)

Tu dis en plus que le problème vient de la création du template où il est proposé Sous-entité Oui/Non.
Ce n'est justement pas un problème pour les éléments qu peuvent être récursifs les temapltes peuvent aussi l'être ce n'est pas un problème (cf; ma première remarque)

Merci de remettre l'état initial, je regarderai ca pour apporter la correction adaptée.

#8 Updated by yllen over 7 years ago

J'étais parti en vrac avec la réponse de Tsmr comme quoi les template ne sont pas récursifs (c'est ta réponse qui m'a mis sur la voie)
J'ai tout reverté et tout retester et je me suis aperçue que le problème n'apparaissait que depuis la 0.84.
Donc en cherchant, je pense avoir enfin trouvé le problème (enfin maintenant ça marche sans modifié la fonction générique) dans mon dernier commit.
J'attends un peu pour faire le backport en 0.85

#9 Updated by moyo over 7 years ago

Pas certain que ca corrige le problème en profondeur.
Les networkequipment étant dans le même cas une correction juste sur les contrats me semble vraiment bizarre.
Je regarderai ca demain si je peux.

#10 Updated by yllen over 7 years ago

Je pense avoir trouvé la solution qui corrige contract et networkEquipment sans changer chaque showForm.
Je te laisse controler

#11 Updated by moyo over 7 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 50 to 100

Applied in changeset r22128.

#12 Updated by moyo over 7 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF