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

SAK-51007 Kernel saving a pronunciation in profile causes SQL syntax error ORA-00933 #13322

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

jesusmmp
Copy link
Contributor

@jesusmmp jesusmmp commented Feb 19, 2025

@jesusmmp jesusmmp requested a review from ern February 19, 2025 18:09
@jesusmmp jesusmmp marked this pull request as ready for review February 19, 2025 18:28
@@ -1941,7 +1941,7 @@ public void removeResource(ContentResourceEdit edit, boolean removeContent)
String filePath = ((BaseResourceEdit) edit).m_filePath;
log.info("Removing resource ("+edit.getId()+") content: "+bodyPath+" file:"+filePath);

String statement = "SELECT COUNT(FILE_PATH) FROM "+resourceTableName+" WHERE FILE_PATH = ?;";
String statement = "SELECT COUNT(FILE_PATH) FROM "+resourceTableName+" WHERE FILE_PATH = ?";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jesusmmp can we also move this statement into the SqlService?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course! I'll fix it tomorrow

@ern ern changed the title SAK-51007 PROFILE - ORA-00933: comando SQL no terminado correctamente - ORACLE SAK-51007 Kernel saving a pronunciation in profile causes SQL syntax error ORA-00933 Feb 20, 2025
@ern ern merged commit bb11609 into sakaiproject:master Feb 20, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants