RTC Release Blocker bug PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar MacJoom
MacJoom
19 Jun 2025

Pull Request for Issue #45484.

Summary of Changes

Remove call to 'toPunycode' in getSafeName which throws error for files with underscores

Testing Instructions

  • Upload files with underscores in the name
  • Upload all kind of files with weird filenames
    File should upload. Special chars should be changed in the uploaded filename

Actual result BEFORE applying this Pull Request

Nothing happens - see Issue

Expected result AFTER applying this Pull Request

File is uploaded

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar MacJoom MacJoom - open - 19 Jun 2025
avatar MacJoom MacJoom - change - 19 Jun 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Jun 2025
Category Front End Plugins
avatar MacJoom MacJoom - change - 19 Jun 2025
Labels Added: PR-6.0-dev
avatar MacJoom MacJoom - change - 19 Jun 2025
The description was changed
avatar MacJoom MacJoom - edited - 19 Jun 2025
avatar MacJoom MacJoom - change - 19 Jun 2025
The description was changed
avatar MacJoom MacJoom - edited - 19 Jun 2025
avatar MacJoom MacJoom - change - 19 Jun 2025
The description was changed
avatar MacJoom MacJoom - edited - 19 Jun 2025
avatar brianteeman
brianteeman - comment - 19 Jun 2025

this PR is not correct. Before this PR if you have a filename using ut8 characters it will be uploaded and cleaned
with this PR the file is simply not uploaded

Test by creating an image with the filename imageäöütest.jpg

Before this pr it will be saved as imageaoutest.jpg

With this pr nothing happens to the image

unrelated bug

avatar brianteeman brianteeman - test_item - 19 Jun 2025 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 19 Jun 2025

I have tested this item 🔴 unsuccessfully on 48f40f8


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

avatar brianteeman brianteeman - test_item - 19 Jun 2025 - Tested successfully
avatar brianteeman
brianteeman - comment - 19 Jun 2025

I have tested this item ✅ successfully on 48f40f8


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

avatar richard67 richard67 - change - 19 Jun 2025
Title
Remove call to 'toPunycode' which throws error for files with '_'
[6.0] Remove call to 'toPunycode' which throws error for files with '_'
avatar richard67 richard67 - edited - 19 Jun 2025
avatar exlemor exlemor - test_item - 19 Jun 2025 - Tested successfully
avatar exlemor
exlemor - comment - 19 Jun 2025

I have tested this item ✅ successfully on 48f40f8

I have test this successfully with: imageäöütest.jpg, acs-delivering_package2_copy.jpg and imageäöütest_pack_test.jpg all were uploaded with appropriate substitution.


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

avatar richard67 richard67 - change - 19 Jun 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 19 Jun 2025

RTC


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

avatar QuyTon QuyTon - change - 20 Jun 2025
Labels Added: RTC Release Blocker bug

Add a Comment

Login with GitHub to post a comment