id,summary,reporter,owner,description,type,status,priority,milestone,component,version,resolution,keywords,cc,drp_resources,i_links,o_links,remaining_time,sprint 2574,"Make ""name"" fields optional in SPW",jamoore,,"Devteam 2010-07-08: {{{ chris: Might have a database issue on PlateAcqusition that needs a fix, one sec. chris 10:07 07:40 ... 07:48 ... 08:02 name | character varying(255) | not null 08:50 Ditto for reagent. 09:02 name | character varying(255) | not null 09:35 Is this one of these ""inherits from 'named'"" issues? josh moore 10:09 yeah. chris 10:09 Same for Plate too. josh moore 10:10 so it's actually not that bad. 10:23 which name's need to be non-nullable on SPW? chris 10:10 The Plate ID and Name attributes are required. ... 10:25 Heh. josh moore 10:10 um, right. chris 10:10 josh moore 10:10 you guys decide that and let me know. chris 10:11 All the attributes are optional. josh moore 10:11 screen as well? chris 10:11 In the schema, yes. josh moore 10:12 seems a bit counter to PDI but whatever. ok. all ""named=true"" are being removed from screen.ome.xml, and replaced with . chris 10:12 I'll probably work around that one though by using the ID if the name doesn't exist. josh moore 10:12 for mage, i'll just fix it by hand. chris 10:12 Having a Screen with no name is going to muck up a ton of stuff. josh moore 10:13 i'd bet jean-marie is expecting a plate.name as well. he usually has code in place to do null ? """" : name, but still. chris 10:13 I just hope that J-M isn't relying on these other names being there, might be a bad idea to change this now. 13:44 During Import I might just insert the object ID if that Name is null. chris 10:18 Working around it in code for now Josh. josh moore 10:28 ok. so we're not changing the DB. i'll add a ticket for 4.2.1 }}} ---- == Decision == * Name should not be optional for Project, Dataset, Image, Screen, Plate. The only exception being !PlateAcquisition * Model: update model.",task,closed,major,OMERO-Beta4.3,ORM,,fixed,,jburel cxallan ajpatterson jrswedlow,,,,0,2011-04-21 (10)