-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Enable to open a file containing an '+' character in its title in the chat application - EXO-64326 #608
Conversation
df1a041
to
a23e3a9
Compare
@@ -188,6 +189,8 @@ public Response persistFile(@Context SecurityContext securityContext, | |||
String workspace = node.getSession().getWorkspace().getName(); | |||
String repository = ((ManageableRepository) node.getSession().getRepository()).getConfiguration().getName(); | |||
String nodePathWithWorkspace = workspace + node.getPath(); | |||
String fileNameFromPath = node.getPath().substring(node.getPath().lastIndexOf("/") + 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't we use node.getName() or if the node title exists, node.getProperty("exo:title").getString() ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR updated !
86274c6
to
e402e51
Compare
Your PR triggers too many exo-ci builds! Please finish your work and then, set your PR ready! Thank you |
a23e3a9
to
ab0e7b9
Compare
…n the chat application - EXO-64326
ab0e7b9
to
67a5b0b
Compare
67a5b0b
to
5384e4b
Compare
Kudos, SonarCloud Quality Gate passed! |
Before to this change , after uploading a file with name containing a '+' character in chat application ,we were unable to open it , the problem that was the '+' character on the file's path being replaced by a space .
After encoding the node name , This change is going to clean the file name using the utils clean name method.