{"action":"created","issue":{"url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/21","repository_url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen","labels_url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/21/comments","events_url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/21/events","html_url":"https://github.com/RISCSoftware/cpacs_tigl_gen/issues/21","id":379786420,"node_id":"MDU6SXNzdWUzNzk3ODY0MjA=","number":21,"title":"Order of CPACS elements in Sequences not correctly represented","user":{"login":"sdeinert","id":15669066,"node_id":"MDQ6VXNlcjE1NjY5MDY2","avatar_url":"https://avatars2.githubusercontent.com/u/15669066?v=4","gravatar_id":"","url":"https://api.github.com/users/sdeinert","html_url":"https://github.com/sdeinert","followers_url":"https://api.github.com/users/sdeinert/followers","following_url":"https://api.github.com/users/sdeinert/following{/other_user}","gists_url":"https://api.github.com/users/sdeinert/gists{/gist_id}","starred_url":"https://api.github.com/users/sdeinert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdeinert/subscriptions","organizations_url":"https://api.github.com/users/sdeinert/orgs","repos_url":"https://api.github.com/users/sdeinert/repos","events_url":"https://api.github.com/users/sdeinert/events{/privacy}","received_events_url":"https://api.github.com/users/sdeinert/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-12T13:53:13Z","updated_at":"2018-11-13T19:33:52Z","closed_at":null,"author_association":"NONE","body":"The writeCPACS methods of e.g. the ribsDefinition class are not created with the order that is prescribed by the Sequence defined in the XSD of the ribsDefinition element. Hence, in this example the child element ribCrosssection is always written before the ribsPositioning/ribExplicitPositioning elements causing validation errors with the schema.\r\n\r\nSequence order should be adhered to in the writeCPACS methods to avoid unnecessary schema conflicts.\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/comments/438406242","html_url":"https://github.com/RISCSoftware/cpacs_tigl_gen/issues/21#issuecomment-438406242","issue_url":"https://api.github.com/repos/RISCSoftware/cpacs_tigl_gen/issues/21","id":438406242,"node_id":"MDEyOklzc3VlQ29tbWVudDQzODQwNjI0Mg==","user":{"login":"bernhardmgruber","id":1224051,"node_id":"MDQ6VXNlcjEyMjQwNTE=","avatar_url":"https://avatars1.githubusercontent.com/u/1224051?v=4","gravatar_id":"","url":"https://api.github.com/users/bernhardmgruber","html_url":"https://github.com/bernhardmgruber","followers_url":"https://api.github.com/users/bernhardmgruber/followers","following_url":"https://api.github.com/users/bernhardmgruber/following{/other_user}","gists_url":"https://api.github.com/users/bernhardmgruber/gists{/gist_id}","starred_url":"https://api.github.com/users/bernhardmgruber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bernhardmgruber/subscriptions","organizations_url":"https://api.github.com/users/bernhardmgruber/orgs","repos_url":"https://api.github.com/users/bernhardmgruber/repos","events_url":"https://api.github.com/users/bernhardmgruber/events{/privacy}","received_events_url":"https://api.github.com/users/bernhardmgruber/received_events","type":"User","site_admin":false},"created_at":"2018-11-13T19:33:52Z","updated_at":"2018-11-13T19:33:52Z","author_association":"MEMBER","body":"This problems actually already affects the type system built by the generator, as ReadCPACS and members of the generated class are in the wrong order as well. But I believe the problem is not too hard to solve. The child element iterating in SchemaParser.cpp has to be adapted to not e.g. iterate over all elements in a sequence and then over all choices. It has to iterate once over all children (in order) and do the appropriate thing.\r\n\r\n@RlanderRISCSW do you want to fix this? Maybe I have time within the next weeks."}}
{"actor":{"display_login":"bernhardmgruber"}}