Disallow setting of core construct definition properties when idref is set #175

Activity

1. bworrell

   added this to the v1.1.1.2 milestone on Aug 14, 2014
2. imjonsnooow commented on Aug 28, 2014

   

   imjonsnooow

   on Aug 28, 2014

   Contributor

   More actions

   Question: should we go with a 'set it and forget it' method, or do we want to actively prohibit setting properties when idref is set? The easiest solution would be to check if idref is set in API objects' to_obj method, and not convert properties if idref is set. However, if we want to throw errors in these conditions, we'll have to modify all of the setters for all of the objects.

   Any preference as to which route we should go? I'm guessing we'll want to go with the latter, so people aren't confused when they try to set properties and those properties are silently dropped...
3. bworrell

   self-assigned this

   on Oct 14, 2014
4. bworrell

   modified the milestone: v1.1.1.2 on Dec 4, 2014