? Language Change NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
5 Jul 2022

Pull Request for Issue #37575 .

Summary of Changes

Display an error message with text not just an empty error

Testing Instructions

npm run build:js
or install prebuilt package

Go to the package installer and try to drag & drop a folder

Actual result BEFORE applying this Pull Request

image

and js error in the console

Expected result AFTER applying this Pull Request

image

and no console errors

Documentation Changes Required

none

avatar brianteeman brianteeman - open - 5 Jul 2022
avatar brianteeman brianteeman - change - 5 Jul 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jul 2022
Category Administration Language & Strings JavaScript Repository NPM Change
avatar richard67
richard67 - comment - 5 Jul 2022

Unfortunately I cannot reproduce the issue here on a clean, current 4.2-dev without this PR. I don't get an empty error message when trying to upload a folder. I get a message "No Install Type Found". I've tried an empty folder and a folder with one empty file and also a folder with files and subfolders. In all cases the same result. What am I doing wrong?

P.S. The message I get is of type information or has no type. It has blue background.

avatar richard67
richard67 - comment - 5 Jul 2022

were you using drag and drop?

Yes, was on administrator/index.php?option=com_installer&view=install and dragged the folder from my Windows 10 Desktop into the drop area of that page.

avatar brianteeman brianteeman - change - 5 Jul 2022
Labels Added: Language Change NPM Resource Changed ?
avatar richard67
richard67 - comment - 5 Jul 2022

After I've dropped the folder, it shows "Uploading 0%" or something like that for a while, and then:
2022-07-05_1

Client (browser) is Windows, and the webserver is Linux.

avatar richard67
richard67 - comment - 5 Jul 2022

What a pity because I think this PR is right now, and I wanted to test it with success, only I can't reproduce the issue.

avatar Fedik
Fedik - comment - 5 Jul 2022

Maybe some OS specific, I have "empty error" as in description, when doing "folder drag and drop" in Linux in Chrome

avatar brianteeman
brianteeman - comment - 5 Jul 2022

recorded on windows 11

Before

before

avatar richard67
richard67 - comment - 5 Jul 2022

Hmm, no, same result when I use Firefox on my Linux server directly.

avatar richard67
richard67 - comment - 5 Jul 2022

@pearlpriyanka Did you have your issue #37575 with Windows 11 as client OS where the browser was used?

avatar richard67
richard67 - comment - 5 Jul 2022

Hmm, anyway, I believe you guys of course, no need for a video or animated gif. I just can't reproduce it here and would like to know why.

avatar Fedik Fedik - test_item - 5 Jul 2022 - Tested successfully
avatar Fedik
Fedik - comment - 5 Jul 2022

I have tested this item successfully on e57fc28


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38227.

avatar joomdonation joomdonation - test_item - 5 Jul 2022 - Tested successfully
avatar joomdonation
joomdonation - comment - 5 Jul 2022

I have tested this item successfully on e57fc28


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38227.

avatar richard67 richard67 - change - 5 Jul 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 5 Jul 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38227.

avatar Quy Quy - change - 7 Jul 2022
Labels Added: ?
avatar roland-d roland-d - change - 12 Jul 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-07-12 12:15:46
Closed_By roland-d
avatar roland-d roland-d - close - 12 Jul 2022
avatar roland-d roland-d - merge - 12 Jul 2022
avatar roland-d
roland-d - comment - 12 Jul 2022

Thank you. As for the discussion of which mime-types to check, that can be a separate issue.

Add a Comment

Login with GitHub to post a comment