[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [Elist Home]
Subject: Re: [wsrp-wsia] v0.90, comments and questions
Hi Rich, I'm missing request #35, did you send it around? Nevertheless here are my comments on #35: I would agree that we should change it to name=value. I see no real advantage in have it the way it is today. However this would have impacts on differnt document sections. We need to define a parameter name which MUST be present in the parameters field like wsrp-urlType={wsrp-urlType} (add wording). Second the BNF syntax changes (10.2.3). Third we should change the producer URL writing example in 10.2.2. Mit freundlichen Gruessen / best regards, Richard Jacob ______________________________________________________ IBM Lab Boeblingen, Germany Dept.8288, WebSphere Portal Server Development Phone: ++49 7031 16-3469 - Fax: ++49 7031 16-4888 Email: mailto:richard.jacob@de.ibm.com |---------+----------------------------> | | Rich | | | Thompson/Watson/I| | | BM@IBMUS | | | | | | 01/23/2003 03:16 | | | PM | |---------+----------------------------> >--------------------------------------------------------------------------------------------------------------------------------------------------| | | | To: wsrp-wsia <wsrp-wsia@lists.oasis-open.org> | | cc: | | Subject: Re: [wsrp-wsia] v0.90, comments and questions | >--------------------------------------------------------------------------------------------------------------------------------------------------| intertwined with <rt/> Rich Thompson Alejandro Abdelnur <alejandro.abdelnur@sun.com> 01/23/2003 03:58 AM To: wsrp-wsia <wsrp-wsia@lists.oasis-open.org> cc: Subject: [wsrp-wsia] v0.90, comments and questions Hi everybody, I?e made it back. <rt>Good to have you back! Hope it was a great vacation.</rt> First of all, great job with v0.90. The flow of the specification is very easy to follow, concepts are explained in a very clear and simple way. Following some comments and feedback on it. Some of my comments may have been already addressed (I did not have time to catch up with all the email discussions yet). Alejandro <COMMENT#>- P<PAGE#>/L<LINES#> 1- P10/L3 Consumer Configured Portlet it seems a misleading name, I would suggest something denoting that are consumer created entities instead. <rt>Suggestions welcome ... open change request = #4</rt> 2- P10/L19-23 Should we add something like Although this does not imply that the portlet should not validate the End-User input. <rt>added change request #28</rt> 3- P14/L27 Instead this functionality I would explicitly mention bookmarking functionality. At first it gave me the impression it was referring to navigational state. <rt>added change request #29</rt> 4- P16/L29-P17/L9 It appears to me that these two sections are about the same thing. Wouldn? make sense to merge them in one section <rt>added change request #30</rt> 5- P20/L1-3 I need somebody to refresh my memory. When did we agree on 255 as a maximum handle length? <rt>Nov. F2F</rt> 6- P20/L38-P21/L19 Where and what for are we using Resource and ResourceList types? <rt>They are the structures used to carry string values for other locales</rt> 7- P22/L1 MarkupType name for this type is misleading. <rt>I've started to rename it several times, but never came up with a better name ... need a suggestion and then I'll open a change request</rt> 8- P22/L19 The portlet description should a ModelDescription element, shouldn? it? <rt>We had decided not to automatically include this in the PortletDescription (issue #120)</rt> 9- P23/L27-42 If the userContext or the templates are stored in the session and the change within the span of a session, they should be resent, right? If so, we should clarify so. <rt>added change request #31</rt> 10- P24/L19-23 Is this section saying that the value is always and array, and it could be of generic objects or just string? I assume so, but it is not clear to me. <rt>It is trying to say that in general it is an array of objects. Since we expect it to frequently be a single string, an alternate form for passing this simpler type is provided.</rt> 11- P25/L24 ModelDescription does not convey the meaning of this type, I would suggest something like PropertiesDescription. <rt>It is used to describe the model of something. Current two uses are Producer-specific registration data and Portlet-specific properties. 12- P27/L27 What is the purpose of the RegistrationState ? <rt>It is a RegistrationContext without the handle ... suggestions for a better name?</rt> 13- P36/L4-11 What kind of escaping? <rt>added change request #32</rt> 14- P36/L12-16 What is the purpose of markupBinary ? <rt>Carrying markup types that do not map into a string field. Examples include an image and a DIME attachment (using currently proposed mechanisms)</rt> 15- P38/L10-15/L29-30 SessionContext, portletContext and markupContext elements are missing from the UpdateResponse IDL description. <rt>They are encapsulated in the interactionResponse field ... decision was to use this format to clarify that blocking interactions return everything from the non-blocking case. adding change request to delete textual part that was missed (#33).</rt> 16- P40/L15 What is the behavior en the cache has expired, the consumer sends the validateTag to the producer and the producer says it? OK? Is a new expiration sent with the response? It should, right? <rt>added change request #34</rt> 17- P40/L26-33 In the case of a allUsers?cache, the expiration mechanism is not clear. It says that if the MarkupParameter change the cache must be invalidated. Which one? How different locales are handled? Different markups?, etc ? <rt>I read this to say that the markup may be cached and supplied to all users who would be using the same MarkupParameters. MarkupParameters does include locale, markupType, etc.</rt> 18- P42 Not that I fully agree with this (email to follow), but if this is the behavior we should explicitly mention that properties cannot be changed in the getMarkup call. I know that because of the method signature it is not possible, but as a clarification to the reader. <rt>I presume you mean portletState can not be changed in getMarkup(). I would note that this is in the section dealing with Interaction Operations (i.e. 6.3).</rt> 19- P44/L1-15 I don? agree with the recommendation on not swapping from non-secure to secure and vice versa. There are app servers and web servers that support this. The Servlet spec allows implementations to support this. Also, if we do not recommend this swapping, why do we provide the secure url-templates when we are insecure and vice versa? <rt>secure to non-secure would be a security violation. The non-secure to secure is guidance ... if the Consumer "knows" it will work for the Producer, this direction of movement is less problematic.</rt> 20- P48/L17-19 How is solo different from maximized? I mea, practically, what would the consumer or producer do differently? <rt>Examples raised included other portlets appearing minimized in maximized windowState vs not appearing in solo windowState.</rt> 21- P49/L16-17 What does it mean the last sentence? <rt>If the Portlet requires a userCategory to work properly, the Producer should provide a default for when the Consumer does not supply one.</rt> 22- P49/L22-31 I think I understood the reason behind changing roles for user categories, not to imply a security related classification. But Administrator and User, they are security related classification. <rt>see change request #5</rt> 23- P49/L32-24 How is different from a request with no user categories when both consumer and producer support them? I thing the Guest user category is redundant. <rt>Do bring this up in the discussion about change request #5.</rt> 24- P60/L25-42 Could we change the wsrp-urlType syntax so it is name=value as all the other parameters? This would allow implementations that handle the type as a parameter not to have to parse the parameter string. <rt>added change request #35</rt> 25- P65/L7-23 What is the purpose of the interaction parameters being generic to a producer? I don? get it. <rt>Makes it simpler for some Consumers.</rt> ---------------------------------------------------------------- To subscribe or unsubscribe from this elist use the subscription manager: <http://lists.oasis-open.org/ob/adm.pl> ---------------------------------------------------------------- To subscribe or unsubscribe from this elist use the subscription manager: <http://lists.oasis-open.org/ob/adm.pl>
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [Elist Home]
Powered by eList eXpress LLC