+1 for this patch, Eric.

F.

On Mon, Jun 14, 2010 at 4:30 PM, Eric Lemoine
<[email protected]> wrote:
> On Mon, Jun 14, 2010 at 4:08 PM, François Van Der Biest
> <[email protected]> wrote:
>> Hi all,
>>
>> Currently, we have in LayerRecord.js :
>> GeoExt.data.LayerRecord = Ext.data.Record.create([
>>    {name: "layer"},
>>    {name: "title", type: "string", mapping: "name"}
>> ]);
>>
>> The mapping can be annoying, for instance when restoring a WMC context :
>> layerStore.add(wmcReader.readRecords(wmcFormat.read(wmcString, {})).records);
>> In the above case, layerRecords end up with their title set to the
>> layer names instead of their correct titles.
>>
>> A quick workaround is to set values["title"] to layerContext["title"]
>> in WMSReader.js, L102
>> Would you see other options ? (as I don't think that removing the
>> above mapping would be a valid one)
>
> Maybe the WMCReader should override this field, just like the
> WMSCapabilitiesReader does. See the attached patch.
>
>
>
>
> --
> Eric Lemoine
>
> Camptocamp France SAS
> Savoie Technolac, BP 352
> 73377 Le Bourget du Lac, Cedex
>
> Tel : 00 33 4 79 44 44 96
> Mail : [email protected]
> http://www.camptocamp.com
>
_______________________________________________
Dev mailing list
[email protected]
http://www.geoext.org/cgi-bin/mailman/listinfo/dev

Reply via email to