When the DropzoneJS module is installed without the dropzone.js in the correct place, there is no warning. The module just does not work.

Add a warning in the status report.

CommentFileSizeAuthor
#2 library_missing_2627362.patch826 bytescgalli
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cgalli created an issue. See original summary.

cgalli’s picture

cgalli CreditAttribution: cgalli as a volunteer commented
FileSize
library_missing_2627362.patch826 bytes

Added dropzonejs.install with the necessary requirements

cgalli’s picture

cgalli CreditAttribution: cgalli as a volunteer commented
Status: Active » Needs review
slashrsm’s picture

slashrsm CreditAttribution: slashrsm at MD Systems GmbH commented
Issue tags: +Zurich media sprint 2015
Primsi’s picture

Primsi CreditAttribution: Primsi at Examiner.com for Examiner.com commented
Status: Needs review » Needs work 
  1. +++ b/dropzonejs.install
@@ -0,0 +1,26 @@
+  $requirements = array();

    Nitpick, but the new array syntax is preffered [] :) 

  2. +++ b/dropzonejs.install
@@ -0,0 +1,26 @@
+      'title' => t('Dropzone Library missing'),

    library

Otherwise looks good. Thanks

slashrsm’s picture

slashrsm CreditAttribution: slashrsm at MD Systems GmbH commented
Status: Needs work » Fixed

Applied patch, fixed #5 nitpiks and committed it. Thanks!

  • slashrsm committed 5f11953 on 8.x-1.x authored by cgalli 
    Issue #2627362 by cgalli, slashrsm, Primsi: Warning missing when the...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.