Issue Details (XML | Word | Printable)

Key: MDL-19092
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Eric Merrill
Reporter: Eric Merrill
Votes: 0
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Workshop password login page incomplete/broken

Created: 07/May/09 05:28 AM   Updated: 12/May/09 04:47 PM
Component/s: Workshop
Affects Version/s: 1.9.4
Fix Version/s: 1.6.9+, 1.7.7+, 1.8.9, 1.9.5

File Attachments: 1. File workshop.diff (0.7 kB)

Issue Links:
Duplicate
 

Participants: David Mudrak and Eric Merrill
Security Level: None
QA Assignee: David Mudrak
Resolved date: 07/May/09
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE


 Description  « Hide
When a user (student) goes to view a workshop with a password, there are a number of errors on the page that is displayed.

The form isn't closed, incorrect js is used (not compatable with all browsers), if will not function with js turned off, and the page footer is not printed.

The attached diff should address all those problems.

I know workshop is getting a major overhall for 2.0, but it would be nice if this fix could make it in for 1.9.5 - I can commit it if there isn't any objection.

Thanks



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
David Mudrak added a comment - 07/May/09 03:02 PM
Eric,

thanks for the patch. Is JavaScript event onClick needed for the <input> type "submit"? Does it have any special meaning here? I would leave it without JavScript.

Feel free to commit and assign me as QA.

TIA


Eric Merrill added a comment - 07/May/09 08:42 PM
Just got rid of the js, since I agree with David that it appears to have no purpose.

Committed changes to 19, 18, 17, and 16.


David Mudrak added a comment - 12/May/09 04:26 PM
Reviewed. Closing. Thanks Eric!