Moodle

Images not uploaded using picture field

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: 1.9.5
  • Labels:
    None
  • Environment:
    Tested on Debian and Fedora, either PHP4 and PHP5
  • Database:
    MySQL
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

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!"

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

David, would you please find some time to evaluate this?

Show
Petr Škoda (skodak) added a comment - David, would you please find some time to evaluate this?
Hide
David Mudrak added a comment -

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.

Show
David Mudrak added a comment - 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.
Hide
Andrea Bicciolo added a comment -

Tested the David patch from the moodle_19_stable branch. Images are uploaded correctly now.

Show
Andrea Bicciolo added a comment - Tested the David patch from the moodle_19_stable branch. Images are uploaded correctly now.
Hide
Howard Miller added a comment -

MDL-18542 reverted. This solves this problem.

Show
Howard Miller added a comment - MDL-18542 reverted. This solves this problem.
Hide
Petr Škoda (skodak) added a comment -

thanks

Show
Petr Škoda (skodak) added a comment - thanks

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: