Javascript error: Missed Files
Description
Environment
Test Plan
Attachments
depends on
Activity

Hudson CI Server December 10, 2014 at 9:08 PM
SUCCESS: Integrated in sakai-trunk-java-1.7 #565 (See http://builds.sakaiproject.org:8080/job/sakai-trunk-java-1.7/565/)
reverse-merge of r315965. We don't want htmlarea back (ottenhoff@longsight.com: rev 315985)

Sam Ottenhoff December 10, 2014 at 5:11 PM
I reversed merged in r315985.
> I was not sure why these folders were removed, if it was intentional or maybe an error, so I decided to recover them.
It was intentional. HTMLArea is a very old rich-text editor. We now use CKEditor.
> I don't know why calculated questions are using these files
Bad code that was probably copied.
I will create a new JIRA to fix the calculatedQuestion.jsp

Juanjo Meroño December 10, 2014 at 12:02 PM
I was not sure why these folders were removed, if it was intentional or maybe an error, so I decided to recover them. I don't know why calculated questions are using these files, so I think the best way to proceed is file a new jira to remove these old files and its references from samigo, maybe these folders are referenced in other places too. Sounds good?

Sam Ottenhoff December 10, 2014 at 11:22 AM
I don't understand the work on this JIRA. Why add these very old files back instead of attempting to solve the problem?
Why is calculated question including these htmlarea files? Why does calculated question need htmlarea?

Jose Mariano Lujan December 10, 2014 at 1:47 AM
If is merged back to 10.x, this one should also be merged back. Otherwise, the issue is resolved in trunk. Thanks .
Error introduced in SAM-1817. Steps to reproduce:
-Create an assessment
-Add a Calculated question to the assessment.
-The browser console shows the javascript error attached to this Jira (javascript error.png).
There are many files that are not found. The error is introduced in SAM-1817 because it deletes the folders: "samigo-app/src/webapp/htmlarea" and "samigo-app/src/webapp/jsf/widget/wysiwyg".