search

dbarzin / deming

Management tool for the information security management system / Outil de gestion du système de management de la sécurité de l'information
GNU General Public License v3.0
227 stars 55 forks source link

Modification du champ `values` de la table `attributes` #82

Closed AntoninLep closed 4 months ago

AntoninLep commented 5 months ago

Bonjour Didier,

Nous souhaitons initialiser des listes d'attributs qui dépassent la limite actuellement en place (VARCHAR(512)). Serait-il possible d'augmenter la taille du VARCHAR utilisé dans la table attributes ? Ou à défaut, de transformer ce champ en un champ de type TEXT ?

Pour avoir un ordre d'idée, notre plus longue liste d'attributs fait environ 2900 caractères.

dbarzin commented 5 months ago

Le champ va être modifié.

dbarzin commented 5 months ago

Le champ a été modifié en varchar(4096). Pour mettre à jour : 

git pull
php artisan migrate
AntoninLep commented 4 months ago

Je viens de déployer la MàJ sur mon instance, tout est bon, merci pour ta réactivité !

Juste par curiosité, pourquoi ne pas choisir un champ de type TEXT directement, plutôt que d'augmenter la taille à 4096 ?

dbarzin commented 4 months ago

Oui, effectivement, mais en poussant ce raisonnement plus loin, on pourrait aussi utiliser des champs TEXT partout. Donc, bien qu'il n'y ait pas de raison particulière, la taille maximale d'un VARCHAR étant de 65535 caractères, j'ai appliqué le principe KISS et changé la taille de ce champ de 512 à 4096 caractères.

AntoninLep commented 4 months ago

Effectivement, parfois il vaut mieux faire au plus simple, merci pour les précisions :)