Skip to content
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

Copy URL to a file won't work by UUID #77

Closed
peterbraemac opened this issue Jul 3, 2018 · 3 comments
Closed

Copy URL to a file won't work by UUID #77

peterbraemac opened this issue Jul 3, 2018 · 3 comments

Comments

@peterbraemac
Copy link
Contributor

Using the URL to reference a document in OpenKM doesn't work by UUID if the default language is set to something. The issue is present in version CE 6.3.6. See full discussion and suggested fix in the forums here:
https://forum.openkm.com/viewtopic.php?f=5&t=9826&start=15

In order to fix the issue, the file index.jsp needs to be changed. The below line, which is currently line 29:
if (docPath != null || fldPath != null) {

Needs to be changed to:
if (docPath != null || fldPath != null || uuid != null) {

@peterbraemac
Copy link
Contributor Author

Here is the path to the index.jsp file that needs to be updated:
src/main/webapp/index.jsp

@peterbraemac
Copy link
Contributor Author

peterbraemac commented Jul 3, 2018

I am a newbie with GitHub so messed up the change/pull request. Anyway, I tried to create a pull request here:

#78

Note that the second change (at the end of the file) was not on purpose and should not be included (empty line at end of file). Only the if statement should be changed, as explained in the first post of this issue.

@gnujavasergio
Copy link
Contributor

Mergue
#78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants