codeconsole commented on PR #16149: URL: https://github.com/apache/grails-core/pull/16149#issuecomment-5595307567
Done in 5fa7a5e - the `@Deprecated` is off `getCurrentRequest()` and nothing else changed. The renames stay, as you said. The method still returns what `getRequest()` returns, since the multipart substitution that used to make the two differ is gone; that is now just two accessors answering identically rather than one being steered away from. The spec asserted the annotation was present, so it asserts the behaviour instead, and upgrade note 54.5 says the two are the same object rather than telling readers to move off it. `setMultipartRequest(..)` keeps its deprecation - that is the shim you asked for in the earlier round, where the method had been removed outright. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
