[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [cti-cybox] Email Object
Thanks all for the feedback. We’ve gone ahead and split this property (and the similar property in mime-part-type) into two, particularly to address the patterning issues that JMG mentioned. Please review the changes in the document [1] if you get a chance. However, it’s worth noting that this does not negate the need for this particular design pattern (an OR between possible types) in future CybOX Objects. [1] https://docs.google.com/document/d/1oPAHN6nitdVF60RuDlajq0VuN6S_p_RP3ZE48yOBBfQ/edit#heading=h.grboc7sq5514 Regards, Ivan On 9/19/16, 1:02 PM, "cti-cybox@lists.oasis-open.org on behalf of Bret Jordan (CS)" <cti-cybox@lists.oasis-open.org on behalf of Bret_Jordan@symantec.com> wrote: >Based on this, I really think we need to change this. > >Bret > >Sent from my Commodore 64 > >> On Sep 19, 2016, at 12:33 PM, John-Mark Gurney <jmg@newcontext.com> wrote: >> >> Jordan, Bret wrote this message on Thu, Sep 15, 2016 at 14:45 +0000: >>> For the record, I really dislike it when a property can be one of two types depending on the value of some other property. For example, we have the following two properties: >>> >>> "is_multipart" boolean >>> "body" string or list of type mime-part-type >>> >>> So if "is_multipart" is false then "body" is a [string] however, if "is_multipart" is true then "body" is a [list of type mime-part-type]. >>> >>> It seems like a much simpler way of doing this is to have two properties for the body... >>> >>> 1) body >>> 2) body_multipart >>> >>> If you still want the boolean property of is_multipart, that is fine. But lets not overload properties based on the value of another property. PLEASE... >> >> I agree that these should be split. >> >> Overloading types will complicate patterning. How do we handle a pattern >> when the type mismatches? >> >> Such as the following when body is text? >> email-message:body[1].file-name = 'foobar.txt' >> >> -- >> John-Mark >> >> --------------------------------------------------------------------- >> To unsubscribe from this mail list, you must leave the OASIS TC that >> generates this mail. Follow this link to all your TCs in OASIS at: >> https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php >> > >--------------------------------------------------------------------- >To unsubscribe from this mail list, you must leave the OASIS TC that >generates this mail. Follow this link to all your TCs in OASIS at: >https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php >
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]