Skip to content

[4.3][bug] fix 5.2 + debug lang#39652

Merged
obuisard merged 8 commits intojoomla:4.3-devfrom
dgrammatiko:patch-7
Jan 19, 2023
Merged

[4.3][bug] fix 5.2 + debug lang#39652
obuisard merged 8 commits intojoomla:4.3-devfrom
dgrammatiko:patch-7

Conversation

@dgrammatiko
Copy link
Copy Markdown
Contributor

Pull Request for Issue #39651 .

Summary of Changes

Testing Instructions

Check the issue

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

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

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.3-dev labels Jan 16, 2023
@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Jan 16, 2023

warning is also possible

var css = data.loaded[extension][file] ? 'alert-success' : 'alert-warning'

<tr>
    <td>pkg_en-GB.sys</td>
    <td>
        <ul>
            <li class="alert-warning">- /language/en-GB/pkg_en-GB.sys.ini</li>
            <li class="alert-warning">- /language/en-GB/en-GB.pkg_en-GB.sys.ini</li>
        </ul>
    </td>
</tr>

@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Jan 16, 2023

success works
image

dgrammatiko and others added 2 commits January 16, 2023 23:09
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Jan 16, 2023

I have tested this item ✅ successfully on d29ea2b

image

Thanks


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 16, 2023

I have tested this item ✅ successfully on d29ea2b


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 16, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 16, 2023
@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Jan 17, 2023

@dgrammatiko Should this code maybe added to plg_system_debug/widgets/languageFiles/widget.css instead of plg_system_debug/css/debug.css?

@dgrammatiko
Copy link
Copy Markdown
Contributor Author

@heelc29 yeah that seems the right place. I already moved the code there, thanks

@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Jan 17, 2023

I have tested this item ✅ successfully on 4ea06bb

Still works


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

@richard67
Copy link
Copy Markdown
Member

Back to pending. One more human test needed.


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

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 17, 2023
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 17, 2023

I have tested this item ✅ successfully on 4ea06bb


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

@Quy Quy added the RTC This Pull Request is Ready To Commit label Jan 17, 2023
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 17, 2023

RTC


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

@obuisard obuisard added this to the Joomla! 4.3.0 milestone Jan 19, 2023
@obuisard obuisard merged commit 52a2ffe into joomla:4.3-dev Jan 19, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 19, 2023
@obuisard
Copy link
Copy Markdown
Contributor

Thank you Dimitris @dgrammatiko for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants