verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

feat: refs #5720 vehicle nuevos campos #2029

Merged
jgallego merged 2 commits from 5720-poolbancario into dev 2024-02-14 14:02:22 +00:00
Conversation 0 Commits 2 Files Changed 1 +13
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
db/versions/10882-blueRuscus/00-vehicle.sql Normal file
View File

@ -0,0 +1,13 @@
ALTER TABLE vn.vehicle ADD supplierFk int(10) unsigned NULL COMMENT 'supplier from whom the vehicle was purchased';
jgallego marked this conversation as resolved
guillermo commented 2024-02-13 08:02:35 +00:00
Review
Copy Link

Creo que acordamos en la convención SQL que los comentarios eran en español

Creo que acordamos en la convención SQL que los comentarios eran en español
jgallego commented 2024-02-13 12:32:17 +00:00
Review
Copy Link

ací no s'acordà res, per tant, es lliure

ací no s'acordà res, per tant, es lliure
ALTER TABLE vn.vehicle ADD CONSTRAINT vehicle_supplierFk FOREIGN KEY (supplierFk) REFERENCES vn.supplier(id);
ALTER TABLE vn.vehicle ADD import decimal(10,2) NULL;
jgallego marked this conversation as resolved
guillermo commented 2024-02-13 08:06:26 +00:00
Review
Copy Link

Si no le pones el DEFAULT el default es NULL?

Si no le pones el DEFAULT el default es NULL?
jgallego commented 2024-02-13 12:23:32 +00:00
Review
Copy Link

depén de com estiga configurada la BBDD en este cas se li posa que accepte valors nulls el camp

depén de com estiga configurada la BBDD en este cas se li posa que accepte valors nulls el camp
ALTER TABLE vn.vehicle ADD supplierCoolerFk int(10) unsigned NULL;
ALTER TABLE vn.vehicle ADD CONSTRAINT vehicle_supplierCoolerFk FOREIGN KEY (supplierCoolerFk) REFERENCES vn.supplier(id) ON DELETE RESTRICT ON UPDATE CASCADE;
ALTER TABLE vn.vehicle ADD vin varchar(50) CHARACTER SET utf8mb3 COLLATE utf8mb3_unicode_ci DEFAULT NULL NULL;
jgallego marked this conversation as resolved
guillermo commented 2024-02-13 08:05:10 +00:00
Review
Copy Link

Cuando se crea un nuevo campo que no va a tener foreign key, no se pone número: https://wiki.verdnatura.es/index.php/Convenciones_SQL#Data_type

Cuando se crea un nuevo campo que no va a tener foreign key, no se pone número: https://wiki.verdnatura.es/index.php/Convenciones_SQL#Data_type
jgallego commented 2024-02-13 12:31:36 +00:00
Review
Copy Link

ixa part de la wiki ni Juan ni Carlos ni jo l'haviem vist, i a mes no s'ha dit mai, lo correcte es especificar un varchar amb la seua longitud.

ixa part de la wiki ni Juan ni Carlos ni jo l'haviem vist, i a mes no s'ha dit mai, lo correcte es especificar un varchar amb la seua longitud.
ALTER TABLE vn.vehicle ADD bankPolicyFk int(11) NULL;
ALTER TABLE vn.vehicle ADD CONSTRAINT vehicle_bankPolicyFk FOREIGN KEY (bankPolicyFk) REFERENCES vn.bankPolicy(id) ON DELETE RESTRICT ON UPDATE CASCADE;