Compile legacyui module against OpenMRS Platform 3.0.0-SNAPSHOT

[Copilot]

Platform 3.0.0-SNAPSHOT upgrades to Spring 7, Hibernate 6, Jakarta EE 10, and Java 21. This migrates the legacyui module to compile and pass all 366 tests against it.

Jakarta EE migration

• javax.servlet → jakarta.servlet across all Java, JSP, and TLD files
• javax.annotation → jakarta.annotation in tests
• POM: javax.servlet-api/jsp-api → jakarta.servlet-api/jakarta.servlet.jsp-api

Spring 6/7 theme support removal

• Spring 6 removed all theme infrastructure. Added local org.openmrs.web.theme package (Theme, ThemeSource, SimpleTheme, ResourceBundleThemeSource, ThemeChangeInterceptor)
• Created org.openmrs.web.taglib.ThemeTag to replace removed org.springframework.web.servlet.tags.ThemeTag; updated spring.tld

Spring MVC namespace conflict

• Replaced <mvc:view-controller> and <mvc:interceptors> with plain bean definitions — the MVC namespace registers a duplicate SimpleControllerHandlerAdapter that conflicts with @EnableWebMvc in openmrs-core's WebConfig

Incompatible Apache taglibs

• Removed page.tld, request.tld, response.tld — these reference javax.servlet-based classes unavailable at runtime
• Replaced <request:existsParameter name="X"> with <c:if test="${not empty param.X}"> in the two JSPs that used it

PortletTag TryCatchFinally

• PortletTag now implements jakarta.servlet.jsp.tagext.TryCatchFinally — Jakarta JSTL's ImportSupport no longer provides these methods

API migrations

• commons-lang → commons-lang3 (escapeHtml → escapeHtml4, escapeJavaScript → escapeEcmaScript)
• Jackson 1.x (org.codehaus.jackson) → Jackson 2.x (com.fasterxml.jackson)
• WebBindingInitializer.initBinder signature updated for Spring 7
• XSSFilter/XSSRequestWrapper updated for Jakarta Servlet API

Build config

• openmrsPlatformVersion → 3.0.0-SNAPSHOT, Java 21, DWR 3.0.0-SNAPSHOT
• CI matrix: removed Java 8/11/17, kept 21 and 25

Test fixes

• Updated test datasets for schema changes (privilege.creator, scheduler_task_config.creator)
• ConceptFormControllerTest.shouldVoidShortName: use non-Numeric datatype to avoid Hibernate 6 TransientPropertyValueException from ConceptNumeric(Concept) constructor reassigning persistent entity back-references
• ProgramFormControllerTest: added flushSession()/clearSession() between sequential controller calls

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.