yangqiaosheng / libcitygml

Automatically exported from code.google.com/p/libcitygml
GNU Lesser General Public License v2.1
0 stars 0 forks source link

Added support for both a Texture and a Material on a geometry #35

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Parse a CityGML file that has both a texture and a material assigned to the 
same geometry
2.
3.

What is the expected output? What do you see instead?
The CityGML specs allow both a texture and a material on every side of a 
geometry. This would translate in up to 4 appearances per surface. The current 
implementation only remembers one appearance at most.

I changed the code to remember a Texture and Material and added new getters and 
setters, getTexture() and getMaterial() to replace the deprecated 
getAppearance() method. I did *not* add support for different appearances for 
the different sides of a geometry, as the 'isFront' property in the citygml is 
currently ignored. It shouldn't be too hard however to extend the API to 
support this, if needed.

Feel free to include my patch in any form. If you have questions, comments or 
want things changed let me know, maybe I can help out.

What version of the product are you using? On what operating system?
r.126

Please provide any additional information below.

Original issue reported on code.google.com by W.Ya...@gmail.com on 27 Feb 2012 at 4:44

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks a lot. Patch applied in Revision 130.

Original comment by jpouder...@gmail.com on 15 Jun 2012 at 6:27