Moodle

Database activity non-standard filepicker javascript does not include drag and drop upload

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Duplicate
  • Affects Version/s: 2.3
  • Fix Version/s: None
  • Component/s: Database activity module
  • Labels:
    None
  • Testing Instructions:
    Hide
    1. Create a new mod_data instance
    2. Add a picture, file and url field to the database
      1. Click 'add entry'
      2. Check you can drag and drop files into the picture and file fields
      3. Check that pictures and files can be selected via the 'choose a file' button
      4. Check the entry is displayed as expected, once it has been saved (you will need to define a template before you can view the entry)
    3. Repeat with a mod_data with just a 'picture' field
    4. Repeat with a mod_data with just a 'file' field

    Note - drag and drop is supported in Firefox, Chrome, Safari and IE10.
    IE9 and below should display the filepicker elements as before this patch.

    Show
    Create a new mod_data instance Add a picture, file and url field to the database Click 'add entry' Check you can drag and drop files into the picture and file fields Check that pictures and files can be selected via the 'choose a file' button Check the entry is displayed as expected, once it has been saved (you will need to define a template before you can view the entry) Repeat with a mod_data with just a 'picture' field Repeat with a mod_data with just a 'file' field Note - drag and drop is supported in Firefox, Chrome, Safari and IE10. IE9 and below should display the filepicker elements as before this patch.
  • Affected Branches:
    MOODLE_23_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    MDL-32456_database_dndupload
  • Pull Master Diff URL:

Description

The database activity module uses a non-standard filepicker element for uploading pictures.

This means it hasn't picked up the new drag and drop upload code.

I have not investigated enough to work out why it needs to use different code (and whether this it still needs to use separate code).

Issue Links

will be (partly) resolved by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-7660 Unable to delete database activity entry attachment

  • Major - Major loss of function, incorrect output
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul Nijbakker added a comment -

I can confirm that this issue exists in Moodle 2.3.2. Drag and rop does not work in the database activity module and it appears to be causing empty entries that break the list view.

Show
Paul Nijbakker added a comment - I can confirm that this issue exists in Moodle 2.3.2. Drag and rop does not work in the database activity module and it appears to be causing empty entries that break the list view.
Hide
Permalink
Itamar Tzadok added a comment -

That's b/c the module's file and picture fields and respective js have not been updated to include the dndupload enhancement.

Show
Itamar Tzadok added a comment - That's b/c the module's file and picture fields and respective js have not been updated to include the dndupload enhancement.
Hide
Permalink
Davo Smith added a comment -

This patch should enable drag and drop upload to mod_data fields

Show
Davo Smith added a comment - This patch should enable drag and drop upload to mod_data fields
Hide
Permalink
Paul Nijbakker added a comment -

Thanks for the patch Davo,
As far as we can tell, the patch has fixed the problem. We expect that the number of empty entries will now drop, but we noticed that the problem with empty entries (if the database has only a file upload field) breaking the list view still exists.

Show
Paul Nijbakker added a comment - Thanks for the patch Davo, As far as we can tell, the patch has fixed the problem. We expect that the number of empty entries will now drop, but we noticed that the problem with empty entries (if the database has only a file upload field) breaking the list view still exists.
Hide
Permalink
Paul Nijbakker added a comment -

Hi Davo,
We discovered that while the drag and drop upload now works in the databases of our 2.3.3 installation, the Choose a file button, for the traditional upload via the filepicker does not work anymore. This stops IE9 (and older) users from using the database.

Show
Paul Nijbakker added a comment - Hi Davo, We discovered that while the drag and drop upload now works in the databases of our 2.3.3 installation, the Choose a file button, for the traditional upload via the filepicker does not work anymore. This stops IE9 (and older) users from using the database.
Hide
Permalink
Davo Smith added a comment -

Thanks for the update - I'll try again locally and see if I can fix it.

Show
Davo Smith added a comment - Thanks for the update - I'll try again locally and see if I can fix it.
Hide
Permalink
Paul Nijbakker added a comment -

We tried replacing in data.js line 77
M.data_filepicker.instances[client_id].show();
with
M.core_filepicker.instances[client_id].show();

This seems to fix the problem of the Choose a file button not working, but it will need more testing to make sure.

Show
Paul Nijbakker added a comment - We tried replacing in data.js line 77 M.data_filepicker.instances [client_id] .show(); with M.core_filepicker.instances [client_id] .show(); This seems to fix the problem of the Choose a file button not working, but it will need more testing to make sure.
Hide
Permalink
Davo Smith added a comment -

Yes, that would be the correct fix - if you look at line 153, that contains the correct code.

I've updated the patch.

Show
Davo Smith added a comment - Yes, that would be the correct fix - if you look at line 153, that contains the correct code. I've updated the patch.
Hide
Permalink
Damyon Wiese added a comment -

Hi Davo,

Adrians recent patch (master only) MDL-7660 changes the file picker for the file manager which allows deleting of files and drag and drop. This is a master only change as it is an improvement rather than a bug fix. If we did decide to backport to stable branches it would probably be better to backport that issue than to have a different patch for master/stables.

Thanks for working on this - but I think we should close this issue as a duplicate.

Regards, Damyon

Show
Damyon Wiese added a comment - Hi Davo, Adrians recent patch (master only) MDL-7660 changes the file picker for the file manager which allows deleting of files and drag and drop. This is a master only change as it is an improvement rather than a bug fix. If we did decide to backport to stable branches it would probably be better to backport that issue than to have a different patch for master/stables. Thanks for working on this - but I think we should close this issue as a duplicate. Regards, Damyon
Hide
Permalink
Davo Smith added a comment -

Closing as duplicate, as suggested above.

Show
Davo Smith added a comment - Closing as duplicate, as suggested above.
Hide
Permalink
Paul Nijbakker added a comment -

Hi, It may all be very well that this database bug will be fixed in version 2.5, but in the meantime, we have a database that does not work properly. Can't this temporary fix be maintained until the final solution arrives?

Show
Paul Nijbakker added a comment - Hi, It may all be very well that this database bug will be fixed in version 2.5, but in the meantime, we have a database that does not work properly. Can't this temporary fix be maintained until the final solution arrives?

People

Vote (3)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved: