search

lawrizy / W3S-tickets

gestion des incidents
0 stars 0 forks source link

Changement db table w3sys_secteur #9

Open DesaedeleerL opened 10 years ago

DesaedeleerL commented 10 years ago

La table w3sys_secteur n'étant plus nécessaire , il faut la supprimer et réécrire le code adéquat

lawrizy commented 10 years ago

Pourquoi cette table n'est plus necessaire ?

Ridounet commented 10 years ago

En fait au départ on pensait qu'une catégorie pouvait être gérée par plusieurs entreprises et qu'une entreprise pouvait gérer plusieurs catégories (relation 'n à n' donc table de jointure). En fin de compte, une catégorie n'est prise en charge que par une seule entreprise et à ce moment-là, ça devient une relation '1 à n' donc table de jointure inutile (foreign key de l'entreprise dans la catégorie, plus simple)

lawrizy commented 10 years ago

Qui a dit qu'il fallait changer cela?

Ridounet commented 10 years ago

On a lancé aucun changement encore, c'est juste pour info, pour améliorer le code, le mieux serait de supprimer cette table. Maintenant si tu veux qu'on la laisse ben ok, le code est déjà fait en prenant compte de la table secteur, le but de ce bug c'est d'optimiser le code

lawrizy commented 10 years ago

Ok donc ce n'est pas un bug. C'est une amélioration. J'ai changé le label (improvment). Ok pour le moment cela reste dans github à titre informatif, on ne fait que ce qui est sur Trello.

Ridounet commented 10 years ago

Table secteur supprimée et code adapté dans les vues et contrôleurs. Je ne pense pas avoir oublié quoi que ce soit (une recherche ne donne rien concernant un éventuel reste de secteur)

Riduan