On Wed, Sep 30, 2015 at 4:29 PM, Patrick Valsecchi < [email protected]> wrote:
> From Oracle, I see that: > > SELECT OSM_ID,NAME,TYPE,GEOM as GEOM FROM > WWW_DATA.BAHNEN_21781 WHERE (SDO_FILTER(GEOM, :1 , 'mask=anyinteract > querytype=WINDOW') = 'TRUE' OR > SDO_FILTER(GEOM, :2 , 'mask=anyinteract querytype=WINDOW') = 'TRUE' ) > :1 = > MDSYS.SDO_GEOMETRY(2003,21781,NULL,MDSYS.SDO_ELEM_INFO_ARRAY(1,1003,3),MDSYS.SDO_ORDINATE_ARRAY(537749.900708669,151378.70118838394,537761.8306244375,151390.63110415262)) > :2 = > MDSYS.SDO_GEOMETRY(2003,21781,NULL,MDSYS.SDO_ELEM_INFO_ARRAY(1,1003,3),MDSYS.SDO_ORDINATE_ARRAY(537749.900708669,151378.70118838394,537761.8306244375,151390.63110415262)) > > Twice the same BBox. > > So no, not normal. > > When I set a break point in StreamingRenderer#findStyleAttributes, I have > one attribute in "attributes" when it's created. It has attrPath="". It has > been added by sae.visit(rule), deep inside the visitor magic. > > Then, in the "if" I'm patching thinks the default geometry is used, but > not part of the attribute list (that's the part that needs fixing) and > therefore adds the default geometry. So I end up with two attributes: ["", > "GEOM"] > > Further down the execution, when the WHERE part of the query is built, we > iterate the attributes returned by StreamingRenderer#findStyleAttributes, > and for each ones we add the SDO_FILTER with the search BBox. For the one > that has an empty name, I guess there is some logic somewhere to replace it > by the default geometry's name. Hence we end up with twice exactly the same > SDO_FILTER. > > And I can see this behavior regardless of the value I set in "Enable > advanced projection handling". > Ok, then it's not the part that was already beaten to death (the two bboxes added by advanced projection handling when crossing the dateline). Thanks for the explanation, the patch appears to be correct. Can you make a pull request? This will also make Travis run a build on it Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003* Le informazioni contenute in questo messaggio di posta elettronica e/o nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il loro utilizzo è consentito esclusivamente al destinatario del messaggio, per le finalità indicate nel messaggio stesso. Qualora riceviate questo messaggio senza esserne il destinatario, Vi preghiamo cortesemente di darcene notizia via e-mail e di procedere alla distruzione del messaggio stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso, divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od utilizzarlo per finalità diverse, costituisce comportamento contrario ai principi dettati dal D.Lgs. 196/2003. The information in this message and/or attachments, is intended solely for the attention and use of the named addressee(s) and may be confidential or proprietary in nature or covered by the provisions of privacy act (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection Code).Any use not in accord with its purpose, any disclosure, reproduction, copying, distribution, or either dissemination, either whole or partial, is strictly forbidden except previous formal approval of the named addressee(s). If you are not the intended recipient, please contact immediately the sender by telephone, fax or e-mail and delete the information in this message that has been received in error. The sender does not give any warranty or accept liability as the content, accuracy or completeness of sent messages and accepts no responsibility for changes made after they were sent or for other risks which arise as a result of e-mail transmission, viruses, etc. -------------------------------------------------------
------------------------------------------------------------------------------
_______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
