Issue Details (XML | Word | Printable)

Key: MDL-18628
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Howard Miller
Reporter: Andrea Bicciolo
Votes: 0
Watchers: 2
Operations

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

Images not uploaded using picture field

Created: 21/Mar/09 01:36 AM   Updated: 24/Mar/09 08:03 PM
Component/s: Database activity module
Affects Version/s: 1.9.4
Fix Version/s: 1.9.5

Environment: Tested on Debian and Fedora, either PHP4 and PHP5
Issue Links:
Relates
 

Database: MySQL
Participants: Andrea Bicciolo, David Mudrak, Howard Miller and Petr Škoda (skodak)
Security Level: None
QA Assignee: Petr Škoda (skodak)
Resolved date: 24/Mar/09
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE


 Description  « Hide
When using Picture filed type on Database Activity, images are not uploaded.

To reproduce:

1. Create a DB activity
2. Add only a Picture Field
3. Create default templates
4. Now try adding a entry: select a image file on your computer, enter the alternate text and click either "Save and view" or "Save and add another".
5. In both cases Moodle displays the message "You did not fill out any fields!"



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Škoda (skodak) added a comment - 24/Mar/09 02:05 AM
David, would you please find some time to evaluate this?

David Mudrak added a comment - 24/Mar/09 06:31 AM
This is a regression of the MDL-18542 patch commited by Howard recently. I have submit a patch reverting the Howard's work that fixes this issue.
When reverted, I am able to save and edit pictures with no problem.

Andrea Bicciolo added a comment - 24/Mar/09 06:51 PM
Tested the David patch from the moodle_19_stable branch. Images are uploaded correctly now.

Howard Miller added a comment - 24/Mar/09 07:56 PM
MDL-18542 reverted. This solves this problem.

Petr Škoda (skodak) added a comment - 24/Mar/09 08:03 PM
thanks