django-cms / django-filer

File and Image Management Application for django
https://django-filer.readthedocs.io/
Other
1.78k stars 581 forks source link

gulp ci failing #1436

Closed fsbraun closed 1 year ago

fsbraun commented 1 year ago

gulp ci fails with

requireSpaceAfterKeywords: One space required after "if" keyword at /home/runner/work/django-filer/django-filer/filer/static/filer/js/addons/dropzone.init.js :
   164 |        // Handle initialization of the dropzone on dynamic formsets (i.e. Django admin inlines)
   165 |        $(document).on('formset:added', function (ev, row) {
   166 |            if(ev.detail && ev.detail.formsetName) {
----------------------^
   167 |                /*
   168 |                    Django 4.1 changed the event type being fired when adding

requireVarDeclFirst: Variable declarations must be the first statements of a function scope. at /home/runner/work/django-filer/django-filer/filer/static/filer/js/addons/dropzone.init.js :
   179 |                ) - 1;
   180 |                let row_ = document.getElementById(event.detail.formsetName + '-' + rowIdx);
   181 |                var dropzones = $(row_).find(dropzoneSelector)
-----------------------------------------------^
   182 |
   183 |            } else {

requireVarDeclFirst: Variable declarations must be the first statements of a function scope. at /home/runner/work/django-filer/django-filer/filer/static/filer/js/addons/dropzone.init.js :
   182 |
   183 |            } else {
   184 |                var dropzones = $(row).find(dropzoneSelector);
-----------------------------------------------^
   185 |            }
   186 |

3 code style errors found.