Cool, I'll take a stab at it later.
Thanks,
mrg
On Fri, Oct 29, 2010 at 9:39 AM, Andrus Adamchik <andru..bjectstyle.org> wrote:
> Yeah I think we removed "refresh" as a query parameter. This is prolly a leftover. Let's deprecate them (and make sure we provide undeprecated flavor and call it within our code).
>
> Andrus
>
> On Oct 29, 2010, at 4:32 PM, Michael Gentry wrote:
>
>> I was looking at DataContext and am curious why we have "refresh" as a
>> parameter to these methods when they aren't being used:
>>
>>
>> � /**
>> � �* Creates a DataObject from DataRow.
>> � �*
>> � �*..ee DataRow
>> � �*/
>> � public <T extends DataObject> T objectFromDataRow(
>> � � � � � Class<T> objectClass,
>> � � � � � DataRow dataRow,
>> � � � � � boolean refresh) {
>>
>> � � � ObjEntity entity =
>> this.getEntityResolver().lookupObjEntity(objectClass);
>>
>> � � � if (entity == null) {
>> � � � � � throw new CayenneRuntimeException("Unmapped Java class: "
>> + objectClass);
>> � � � }
>>
>> � � � ClassDescriptor descriptor = getEntityResolver().getClassDescriptor(
>> � � � � � � � entity.getName());
>> � � � List<T> list = objectsFromDataRows(descriptor,
>> Collections.singletonList(dataRow));
>> � � � return list.get(0);
>> � }
>>
>> � /**
>> � �* Creates a DataObject from DataRow. This variety of the
>> 'objectFromDataRow' method
>> � �* is normally used for generic classes.
>> � �*
>> � �*..ee DataRow
>> � �*..ince 3.0
>> � �*/
>> � public DataObject objectFromDataRow(
>> � � � � � String entityName,
>> � � � � � DataRow dataRow,
>> � � � � � boolean refresh) {
>>
>> � � � ClassDescriptor descriptor =
>> getEntityResolver().getClassDescriptor(entityName);
>> � � � List<?> list = objectsFromDataRows(descriptor,
>> Collections.singletonList(dataRow));
>>
>> � � � return (DataObject) list.get(0);
>> � }
>>
>>
>> Should we deprecate them?
>>
>> mrg
>>
>
>
This archive was generated by hypermail 2.0.0 : Fri Oct 29 2010 - 13:52:06 UTC