? ? Pending

User tests: Successful: Unsuccessful:

avatar ReLater
ReLater
16 May 2019

Pull Request for Issue #24911

Testing Instructions

Expected and actual result

avatar ReLater ReLater - open - 16 May 2019
avatar ReLater ReLater - change - 16 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2019
Category Administration com_modules
avatar infograf768
infograf768 - comment - 16 May 2019

I have tested this item successfully on 67cfd27

Note: it is also needed for site modules edit if the User Status module is unpublished.


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

avatar infograf768
infograf768 - comment - 16 May 2019

I have tested this item successfully on 67cfd27

Note: it is also needed for site modules edit if the User Status module is unpublished.


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

avatar infograf768 infograf768 - test_item - 16 May 2019 - Tested successfully
avatar ReLater
ReLater - comment - 16 May 2019

Note: it is also needed for site modules edit if the User Status module is unpublished.
I'll provide a pr.

avatar infograf768
infograf768 - comment - 16 May 2019

We may improve this by also taking off the code from the User Status module as it will not be necessary anymore.

avatar infograf768
infograf768 - comment - 16 May 2019

What I meant is that adding it in the edit.php as done here solves the issue for all modules edit, site and admin.

avatar infograf768 infograf768 - change - 16 May 2019
Labels Added: ?
avatar ReLater
ReLater - comment - 16 May 2019

Yes, my #24921 (comment) was nonsense ;-)

avatar infograf768 infograf768 - change - 16 May 2019
Title
administrator module edit. Dropdown Save&Close fails when module User Status deactivated
[4.0] Module edit. Dropdown Save&Close fails when module User Status deactivated
avatar infograf768 infograf768 - edited - 16 May 2019
avatar ReLater
ReLater - comment - 16 May 2019

We may improve this by also taking off the code from the User Status module as it will not be necessary anymore.

Let's do it in another pr after this one has been merged. There are several modules with line
HTMLHelper::_('bootstrap.framework'); in tmpl/default.php but I'm not sure if we need them for output and not just for edit purposes.

I've added a reminder to my ToDos.

avatar SharkyKZ
SharkyKZ - comment - 16 May 2019

This should probably be added to dropdown button layout.

avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2019
Category Administration com_modules Layout
avatar ReLater
ReLater - comment - 16 May 2019

Moved fix to layouts/joomla/toolbar/dropdown.php

avatar SharkyKZ
SharkyKZ - comment - 16 May 2019

I have tested this item successfully on aa70678


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

avatar SharkyKZ
SharkyKZ - comment - 16 May 2019

I have tested this item successfully on aa70678


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

avatar SharkyKZ SharkyKZ - test_item - 16 May 2019 - Tested successfully
avatar Quy
Quy - comment - 16 May 2019

I have tested this item successfully on aa70678


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

avatar Quy Quy - test_item - 16 May 2019 - Tested successfully
avatar Quy Quy - change - 16 May 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 16 May 2019

RTC


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

avatar Quy
Quy - comment - 16 May 2019

RTC


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

avatar ReLater ReLater - change - 16 May 2019
Labels Added: ?
avatar wilsonge wilsonge - change - 16 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-16 18:00:32
Closed_By wilsonge
avatar wilsonge wilsonge - close - 16 May 2019
avatar wilsonge wilsonge - merge - 16 May 2019
avatar wilsonge
wilsonge - comment - 16 May 2019

Thanks!

Add a Comment

Login with GitHub to post a comment