Good argument! ;-)

regards,
Armin
----- Original Message -----
From: "Jakob Braeuchi" <[EMAIL PROTECTED]>
To: "OJB Users List" <[EMAIL PROTECTED]>
Sent: Friday, May 30, 2003 6:04 PM
Subject: Re: OJB is updating read-only columns


> hi armin,
>
> readonly test works for me if the fileds are declared as readonly ;)
in
> the repository
>
>         <field-descriptor
>             name="readonlyString"
>             column="RO_STRING"
>             jdbc-type="VARCHAR"
>             access="readonly"
>         />
>
>         <field-descriptor
>             name="readonlyLong"
>             column="RO_LONG"
>             jdbc-type="BIGINT"
>             access="readonly"
>          />
>
> jakob
>
> Armin Waibel wrote:
>
> >Great!
> >----- Original Message -----
> >From: "David Warnock" <[EMAIL PROTECTED]>
> >To: "OJB Users List" <[EMAIL PROTECTED]>
> >Sent: Friday, May 30, 2003 11:59 AM
> >Subject: Re: OJB is updating read-only columns
> >
> >
> >
> >
> >>Armin,
> >>
> >>
> >>
> >>>seems like a bug. I will write a
> >>>test case to check this out.
> >>>
> >>>
> >>I am certain it is a bug. In looking at it I have realised that my
> >>earlier patch which added support for update-lock="false" was
> >>
> >>
> >incomplete.
> >
> >
> >>Both access="readonly" and update-lock="false" need to be checked in
> >>
> >>
> >the
> >
> >
> >>various Sql*Statement classes in org.ojb.broker.accesslayer.sql so
> >>
> >>
> >that
> >
> >
> >>these columns are not added to the prepared statements.
> >>
> >>We may also need to add a check in the code that populates the
> >>parameters for the prepared statements.
> >>
> >>I will submit a patch for this later today.
> >>
> >>Regards
> >>
> >>David
> >>--
> >>David Warnock, Sundayta Ltd. http://www.sundayta.com
> >>iDocSys for Document Management. VisibleResults for Fundraising.
> >>Development and Hosting of Web Applications and Sites.
> >>
> >>
> >>
>
>>---------------------------------------------------------------------
> >>To unsubscribe, e-mail: [EMAIL PROTECTED]
> >>For additional commands, e-mail: [EMAIL PROTECTED]
> >>
> >>
> >>
> >>
> >>
> >
> >
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: [EMAIL PROTECTED]
> >For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
> >
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to