Re: [sca-j] ISSUE 111: Proposed resolution

[Simon Nash <oasis@cjnash.com>]

Mike,
Thanks for catching this omission from my proposed resolution for JAVA-111.
Here's a revised proposal that includes this, and also corrects a line
numbering problem.

The following line numbers refer to sca-javacaa-1.1-spec-cd01-rev2.pdf.

1. Delete line 1044.

2. On line 1048, add the following sentence: "If this constructor has
    parameters, each of these parameters MUST have either a @Property
    annotation or a @Reference annotation."

3. Delete lines 1049 through 1052.

4. Replace line 1062 by:
    public HelloServiceImpl (@Property(name="someProperty") String someProperty) {

5. Replace lines 1381 through 1382 by ". name (optional) - the name of
    the property.  For a field annotation, the default is the name of the
    field of the Java class.  For a setter method annotation, the default
    is the JavaBeans property name corresponding to the setter method
    name.  For a constructor parameter annotation, there is no default
    and the name attribute MUST be present."

6. On line 1383, add "For a constructor parameter annotation, this
    attribute MUST have the value true."

7. Replace lines 1441 through 1442 by ". name (optional) - the name of
    the reference.  For a field annotation, the default is the name of the
    field of the Java class.  For a setter method annotation, the default
    is the JavaBeans property name corresponding to the setter method
    name.  For a constructor parameter annotation, there is no default
    and the name attribute MUST be present."

8. On line 1443, add "For a constructor parameter annotation, this
    attribute MUST have the value true."

9. Replace line 1453 by:
    public void setStockQuote( StockQuoteService theSQService ) {
        ....
    }

   Simon

Mike Edwards wrote:
> 
> Folks,
> 
> +1 to this proposal.
> 
> Can I ask if there is going to be a change to the @Constructor 
> annotation definition which will place a requirement
> for there to be either @Property or @Reference annotations on each 
> parameter of the constructor which is annotated
> with @Constructor?  If we don't require this, then I don't see where the 
> name of the reference or property can come from
> since the names of the parameters are not available at runtime.
> 
> If there is going to be a change for @Constructor, which Issue will do 
> this?
> 
> Yours,  Mike.
> 
> Strategist - Emerging Technologies, SCA & SDO.
> Co Chair OASIS SCA Assembly TC.
> IBM Hursley Park, Mail Point 146, Winchester, SO21 2JN, Great Britain.
> Phone & FAX: +44-1962-818014    Mobile: +44-7802-467431  
> Email:  mike_edwards@uk.ibm.com
> 
> 
> From: 	Simon Nash <oasis@cjnash.com>
> To: 	sca-j@lists.oasis-open.org
> Date: 	15/12/2008 10:22
> Subject: 	[sca-j] ISSUE 111: Proposed resolution
> 
> 
> ------------------------------------------------------------------------
> 
> 
> 
> The following line numbers refer to sca-javacaa-1.1-spec-cd01-rev2.pdf.
> 
> 1. Delete line 1044.
> 
> 2. Delete lines 1049 through 1052.
> 
> 3. Replace line 1062 by:
>    public HelloServiceImpl (@Property(name="someProperty") String 
> someProperty) {
> 
> 4. Replace lines 1344 through 1345 by ". name (optional) - the name of
>    the property.  For a field annotation, the default is the name of the
>    field of the Java class.  For a setter method annotation, the default
>    is the JavaBeans property name corresponding to the setter method
>    name.  For a constructor parameter annotation, there is no default
>    and the name attribute MUST be present."
> 
> 5. Replace lines 1441 through 1442 by ". name (optional) - the name of
>    the reference.  For a field annotation, the default is the name of the
>    field of the Java class.  For a setter method annotation, the default
>    is the JavaBeans property name corresponding to the setter method
>    name.  For a constructor parameter annotation, there is no default
>    and the name attribute MUST be present."
> 
> 6. Replace line 1453 by:
>    public void setStockQuote( StockQuoteService theSQService ) {
>        ....
>    }
> 
> 
> ---------------------------------------------------------------------
> 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
> 
> 
> 
> 
> 
> 
> ------------------------------------------------------------------------
> 
> /
> /
> 
> /Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number 
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU/
> 
> 
> 
> 
> 
>