[OpenNMS/opennms] a48003: NMS-18217: Remove legacy Node List page (#8761)
Scott Theleman via opennms-cvs <[email protected]>
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/release-36.x/[email protected]> |
Branch: refs/heads/release-36.x
Home: https://github.com/OpenNMS/opennms
Commit: a480037d3ce4fbcd3a40563c80c0021b8f1c403e
https://github.com/OpenNMS/opennms/commit/a480037d3ce4fbcd3a40563c80c0021b8f1c403e
Author: Scott Theleman <[email protected]>
Date: 2026-08-07 (Fri, 07 Aug 2026)
Changed paths:
M core/web-assets/src/main/assets/style/opennms-theme.scss
M features/springframework-security/src/test/java/org/opennms/web/springframework/security/LoginModuleUtilsTest.java
M opennms-services/src/test/resources/opennmsPageSequence.xml
R opennms-web-api/src/main/java/org/opennms/web/svclayer/NodeListService.java
R opennms-web-api/src/main/java/org/opennms/web/svclayer/model/NodeListCommand.java
R opennms-web-api/src/main/java/org/opennms/web/svclayer/model/NodeListModel.java
R opennms-web-api/src/main/java/org/opennms/web/svclayer/support/DefaultNodeListService.java
M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeRestService.java
M opennms-webapp-rest/src/main/webapp/WEB-INF/menu/menu-template-alt.json
M opennms-webapp-rest/src/main/webapp/WEB-INF/menu/menu-template-default.json
M opennms-webapp-rest/src/main/webapp/WEB-INF/menu/menu-template-legacy.json
M opennms-webapp-rest/src/main/webapp/WEB-INF/menu/menu-template.json
M opennms-webapp-rest/src/test/java/org/opennms/web/rest/support/menu/MenuProviderTest.java
M opennms-webapp-rest/src/test/resources/dispatcher-servlet.xml
M opennms-webapp-rest/src/test/resources/menu/menu-template.json
R opennms-webapp/src/main/java/org/opennms/web/controller/node/NodeListController.java
M opennms-webapp/src/main/java/org/opennms/web/element/ElementIdNotFoundException.java
M opennms-webapp/src/main/java/org/opennms/web/element/ElementUtil.java
M opennms-webapp/src/main/resources/org/opennms/web/svclayer/applicationContext-svclayer.xml
M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml
R opennms-webapp/src/main/webapp/WEB-INF/jsp/element/nodeList.jsp
R opennms-webapp/src/main/webapp/WEB-INF/tags/element/nodelist.tag
R opennms-webapp/src/main/webapp/WEB-INF/tags/element/pagedList.tag
M smoke-test/src/main/java/org/opennms/smoketest/selenium/AbstractOpenNMSSeleniumHelper.java
M smoke-test/src/test/java/org/opennms/smoketest/AdminPasswordGateIT.java
M smoke-test/src/test/java/org/opennms/smoketest/MenuHeaderIT.java
R smoke-test/src/test/java/org/opennms/smoketest/NodeListPageIT.java
M smoke-test/src/test/java/org/opennms/smoketest/ProvisioningNewUIIT.java
M smoke-test/src/test/java/org/opennms/smoketest/ui/NodeListIT.java
M ui/src/components/Nodes/hooks/useInterfaceListing.ts
M ui/src/styles/opennms-styles.scss
M ui/tests/components/Menu/menu-template-test.json
M ui/tests/components/Menu/utils.test.ts
Log Message:
-----------
NMS-18217: Remove legacy Node List page (#8761)
* NMS-18217: remove legacy JSP node list page and DefaultNodeListService
* NMS-18217: update smoke tests for legacy node list removal
* NMS-18217: fix remaining smoke tests and stale comments referencing removed node list
* NMS-18217: address PR review feedback
- Drop the dead nodePage() call in ProvisioningNewUIIT (immediately
overwritten by direct node-detail navigation) and remove the now
caller-less nodePage() helper
- Replace the last /element/nodeList.htm reference (stale example URL
in LoginModuleUtilsTest) with a surviving page
- Fix pre-existing ElementIdNotFoundException bug: the 6-arg
constructor accepted browseUri but never assigned it, so the
'browse the node list' link on the element-id-not-found error page
never rendered for those call sites
* NMS-18217: force light color-scheme for native checkboxes and radios on legacy pages
Extends the existing <select> fix: in dark mode PrimeVue sets
color-scheme: dark on the document, making native form controls render
dark on the light-only legacy JSP pages (e.g. the alarm list's Ack
checkboxes).
To unsubscribe from these emails, change your notification settings at https://github.com/OpenNMS/opennms/settings/notifications
_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list
To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs