Hi, I updated the patch (already had Ext.urlEncode version there, ive just commented it out)
Can you please check out ticket/311 also and give opinion/suggestions? Thank you®ards, Ivan On Fri, Jul 23, 2010 at 11:56 PM, Andreas Hocevar <[email protected]> wrote: > On Jul 22, 2010, at 10:50 , Andreas Hocevar wrote: > >> On Jul 22, 2010, at 10:39 , Ivan Grcic wrote: >> >>> Hi Andreas, >>> >>> sure thing Ill contribute the patch right away. >>> >>> A question, is it OK to use Ext.urlEncode? >>> >>> url = Ext.urlEncode(baseParams, url); >> >> I'd say go with OpenLayers.Util.urlEncode for now, unless you also want to >> change the examples that use WMSLegend to use Ext 3.2.1. Whatever works best >> for you. >> > > Sorry, I thought Ext.urlAppend. Using Ext.urlEncode is fine, it was already > available in Ext 2.2.1. If you can, please update your patch for > https://trac.geoext.org/ticket/308, otherwise let me know and I'll do it. > > Regards, > Andreas. > > > -- > Andreas Hocevar > OpenGeo - http://opengeo.org/ > Expert service straight from the developers. > > -- Ivan Grcic _______________________________________________ Dev mailing list [email protected] http://www.geoext.org/cgi-bin/mailman/listinfo/dev
