Re: [xwiki-devs] [Notice] Going to merge changes related to URL parsing - Need opinion on API removal

Hi devs, I've worked a bit on the xwiki-platform-url module again and I've refactored more of the XWiki class to use it. I would like to commit this: * Changes for platform: https://github.com/xwiki/xwiki-platform/commit/1505a9030084 * Changes for enterprise: https://github.com/xwiki/xwiki-enterprise/commit/f3b14e42c5a8 In doing so I've removed several public methods from com.xpn.xwiki.XWiki (the internal XWiki class): * public XWikiDocument getDocumentFromPath(String path, XWikiContext context) throws XWikiException * public DocumentReference getDocumentReferenceFromPath(String path, XWikiContext context) * public String getDocumentNameFromPath(String path, XWikiContext context) * public String getDocumentName(XWikiRequest request, XWikiContext context) These are no longer needed since the xwiki-platform-url module provides better replacements. Now even if the removal of these methods don't break CLIRR (since com.xpn.xwiki.XWiki is internal) we need to decide if it's ok to just remove them or if I should move them to the legacy module. I'm asking because it's quite some work (as the previous code relied on classes from the xwiki-platform-url module that don't exist anymore and I would need to also move the code from these removed classes to the legacy module to ensure the same behavior as before). All in all I probably need 1 to 2 days to put all that in the legacy modules. WDYT? Should I do it or is it acceptable that I don't? Note that this is for 5.1M1. Thanks -Vincent