Follow-up to #2566311: JSMin Compression destroy russian characters

The commit from #2566311: JSMin Compression destroy russian characters got already committed with issue #2568381: Gzip failing messages on nginx proxy environment and resulted in duplicate code.

CommentFileSizeAuthor
#2 jsmin_compression-remove-duplicate-code-2608724-2.patch1013 bytesosopolar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

osopolar created an issue. See original summary.

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
CreditAttribution: osopolar commented
FileSize
jsmin_compression-remove-duplicate-code-2608724-2.patch1013 bytes

Status: Needs review » Needs work

The last submitted patch, 2: jsmin_compression-remove-duplicate-code-2608724-2.patch, failed testing.

  • mikeytown2 committed 2f652f0 on authored by osopolar 
    Issue #2608724 by osopolar: Duplicated code as patch form #2566311 got...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs work » Fixed

Thanks for the patch it's been committed. Not sure why the testbot barfed on it.

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
CreditAttribution: osopolar commented

Wow, this was fast, thanks.

Status: Fixed » Closed (fixed)

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