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
4893: DB insert syntax error whilst importing IMS CC Forum in DAO.execute() #12
base: master
Are you sure you want to change the base?
Conversation
continue; | ||
} | ||
if ($error->level==LIBXML_ERR_WARNING) continue; | ||
elseif (strpos($error->message, 'metaMetadata')) continue; |
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 are we skipping metaMetadata in the error message?
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.
Good catch! This was an error that was causing an import failure due to the OpenLearn package using metaMetadata when the schema has minOccurences=0 maxOccurences=0 which means that the use of the element in a document is illegal. I accidentally left this change in after solving the primary issue I committed changes for.
Do you mean that you intend to remove the left-in checking on "metaMetadata"? Thanks. |
I've reverted the change in the additional commit. |
TJ greg Greg Gay |
On Tue, 2011-11-15 at 06:14 -0800, Greg Gay wrote:
Hi Greg. Sure, that'd be fine. We'll be making another push into the TJ. |
TJ, Hello. According to the email below from Greg at ATutor, you may be If you don't have Skype, then we can use email. Thanks and I look forward to hearing back from you. John Bovey Hi John greg -------- Original Message -------- To: Greg Gay ggay@ocad.ca On Tue, 2011-11-15 at 06:14 -0800, Greg Gay wrote:
Hi Greg. Sure, that'd be fine. We'll be making another push into the TJ. Reply to this email directly or view it on GitHub: No virus found in this message. |
see http://atutor.ca/atutor/mantis/view.php?id=4893