[YANGTOOLS-265] Fix bended implementation of handleUnknownNode in YangParserListenerImpl Created: 13/Aug/14 Updated: 10/Apr/22 Due: 13/Aug/15 Resolved: 09/Nov/15 |
|
| Status: | Resolved |
| Project: | yangtools |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | ||
| Reporter: | Lukas Sedlak | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||
| Description |
|
Currently the YangParserListenerImpl is implemented in way that if yang model contains extensions (ie unknown nodes) and extensions contains arguments and value of the extension argument is sperated by ":" the argument value is evaluated as prefix:value. The worst thing is that during parsing of such arguments the parser expects that "prefix" part of argument is real prefix and should be defined within import statements in yang model. The evaluation of extension argumenst in this way works only for config subsystem but can severely crash parsing in case when yang model contain extension which represents mapping in extension argument as key:value etc and does not contain any import definition. Currently this is handled by workaround done by fix: https://git.opendaylight.org/gerrit/#/c/9808/ |
| Comments |
| Comment by Lukas Sedlak [ 13/Aug/14 ] |
|
This depends on: https://bugs.opendaylight.org/show_bug.cgi?id=1538 |
| Comment by Peter Kajsa [ 29/May/15 ] |
|
Will be fixed in new parser impl. |
| Comment by Martin Ciglan [ 31/Jul/15 ] |
|
Per implementation of new statement parser, argument is given as String, which can be parsed as client needs. Based on discussion with Maros Marsalek (netconf, config-sub), this is not an issue anymore on their side. relevant patches pushed: |