? ? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
16 Sep 2020

Pull Request for Issue #30649.

Summary of Changes

Corrects module name.

Testing Instructions

Code review.

Or test on Apache with module mod_autoindex.c enabled:

Rename htaccess.txt to .htaccess.
Create an empty directory or remove index.html from existing directory.
Open directory in browser.

Actual result BEFORE applying this Pull Request

Directory contents listed.

Expected result AFTER applying this Pull Request

Directory contents not listed.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 16 Sep 2020
avatar SharkyKZ SharkyKZ - change - 16 Sep 2020
Status New Pending
avatar SharkyKZ SharkyKZ - change - 16 Sep 2020
The description was changed
avatar SharkyKZ SharkyKZ - edited - 16 Sep 2020
avatar richard67 richard67 - test_item - 16 Sep 2020 - Tested successfully
avatar richard67
richard67 - comment - 16 Sep 2020

I have tested this item successfully on 8e0a67b


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

avatar richard67 richard67 - change - 16 Sep 2020
The description was changed
avatar richard67 richard67 - edited - 16 Sep 2020
avatar ChristineWk ChristineWk - test_item - 16 Sep 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 16 Sep 2020

I have tested this item successfully on 8e0a67b


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

avatar richard67 richard67 - change - 16 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 16 Sep 2020

RTC


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

avatar richard67 richard67 - change - 16 Sep 2020
Labels Added: ? ?
avatar sandewt
sandewt - comment - 16 Sep 2020

Question: will this change also be mentioned in the next update of Joomla in the Post-installation Messages for Joomla CMS?


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

avatar richard67
richard67 - comment - 16 Sep 2020

Question: will this change also be mentioned in the next update of Joomla in the Post-installation Messages for Joomla CMS?

No idea. Ping @HLeithner @zero-24 .

avatar HLeithner
HLeithner - comment - 16 Sep 2020

Only if @SharkyKZ creates a post installation message, alternative we can write it into the release article.

@SniperSister any objections on this.

avatar sandewt
sandewt - comment - 16 Sep 2020

Only if @SharkyKZ creates a post installation message, alternative we can write it into the release article.

As a reminder, joomla 3.9.21 also included a post-install message to change the .htaccess file.


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

avatar SniperSister
SniperSister - comment - 16 Sep 2020

Fine for me!

avatar SharkyKZ SharkyKZ - change - 16 Sep 2020
Status Ready to Commit Pending
avatar SharkyKZ
SharkyKZ - comment - 16 Sep 2020

I'll look into including a postinstall message. Removing RTC now.


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

avatar SharkyKZ SharkyKZ - change - 16 Sep 2020
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 16 Sep 2020
Category SQL Administration com_admin Postgresql MS SQL Language & Strings
avatar SharkyKZ SharkyKZ - change - 16 Sep 2020
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 16 Sep 2020

Postinstall message added.

avatar sandewt sandewt - test_item - 16 Sep 2020 - Tested successfully
avatar sandewt
sandewt - comment - 16 Sep 2020

I have tested this item successfully on 941fce5

Joomla! 3.9.22-dev+pr.30655 Development [ Amani ] 26-August-2020 08:12 GMT
mysql (5.5.5-10.4.14-MariaDB)

Tested:
mod_autoindex.c
and
post install message

Questing: why this addition as this file can not be updated automatically ?

[EDIT]


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30655.
avatar SharkyKZ
SharkyKZ - comment - 17 Sep 2020

Questing: why this addition as this file can not be updated automatically ?

I just copied this from the other message. We don't modify existing .htaccess file, users have to do it themselves.

avatar sandewt
sandewt - comment - 17 Sep 2020

As a reminder, joomla 3.9.21 also included a post-install message to change the .htaccess file.

This message has now disappeared in Joomla 3.9.22 is that the intention?

avatar SharkyKZ
SharkyKZ - comment - 17 Sep 2020

The 3.9.21 message is added only when updating to 3.9.21. It is not shown on new installations or when running on git clone or updating from nightly to nightly. You'd have to manually to run the SQL script to have it added to the database.

avatar sandewt
sandewt - comment - 17 Sep 2020

@SharkyKZ thank you for the explanation.

avatar richard67
richard67 - comment - 18 Sep 2020

@SharkyKZ Postinstall message is not described in summary of changes and testing incstructions.

avatar richard67 richard67 - test_item - 19 Sep 2020 - Tested successfully
avatar richard67
richard67 - comment - 19 Sep 2020

I have tested this item successfully on 8176b37


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

avatar richard67 richard67 - alter_testresult - 19 Sep 2020 - sandewt: Tested successfully
avatar richard67
richard67 - comment - 19 Sep 2020

Previous test by @sandewt was still valid because change after that was only a small improvement of the language string. I've restored the test in the issue tracker so it is counted correctly in the tracker and here.

avatar richard67 richard67 - change - 19 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 19 Sep 2020

RTC


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

avatar HLeithner HLeithner - change - 19 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-19 14:52:19
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 19 Sep 2020
avatar HLeithner HLeithner - merge - 19 Sep 2020
avatar HLeithner
HLeithner - comment - 19 Sep 2020

Thanks

Add a Comment

Login with GitHub to post a comment