[NETCONF-998] OpenAPI: Do not use Optional type as method parameter Created: 24/Apr/23 Updated: 14/Jul/23 Resolved: 14/Jul/23 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | restconf-openapi |
| Affects Version/s: | None |
| Fix Version/s: | 7.0.0 |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Ivan Hrasko | Assignee: | Šimon Ukuš |
| Resolution: | Done | Votes: | 1 |
| Labels: | pt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
In sal-rest-docgen package there is a lot of usages of Optional<> type as method parameter. That’s discouraged: https://stackoverflow.com/questions/31922866/why-should-java-8s-optional-not-be-used-in-arguments. We can get rid of this problem by slightly reworking our code. Complete work on https://git.opendaylight.org/gerrit/c/netconf/+/105646 and create additional patches which deals with other occurrences of this problem in sal-rest-docgen package. |