? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
6 Mar 2022

Pull Request for Issue Page of errors when trying to load deleted template file. #29345

Summary of Changes

Check template exists before running off and attempting anything else

Testing Instructions

Try to edit a template that has been deleted using a bookmark or cached url in your browser (or try to hack yourself by loading an invalid template id) .

The easiest way to replicate is to go to edit the templates files

http://example.com/administrator/index.php?option=com_templates&view=templates

Select administrator

Click Atum Details and Files

gets you a url like this

https://example.com/administrator/index.php?option=com_templates&view=template&id=210&file=aG9tZQ==

Change 210 to lets say 666

Actual result BEFORE applying this Pull Request

A page of errors or more recently just bool

Screen Shot 2022-03-06 at 22 09 03

Expected result AFTER applying this Pull Request

Screen Shot 2022-03-06 at 21 44 59

Documentation Changes Required

none

avatar PhilETaylor PhilETaylor - open - 6 Mar 2022
avatar PhilETaylor PhilETaylor - change - 6 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Mar 2022
Category Administration com_templates
avatar PhilETaylor PhilETaylor - change - 6 Mar 2022
Title
Page of errors when trying to load deleted template file
[4] Page of errors when trying to load deleted template file
avatar PhilETaylor PhilETaylor - edited - 6 Mar 2022
avatar PhilETaylor
PhilETaylor - comment - 6 Mar 2022

You totally miss the point.

What you are showing is a correct 404 page with debug mode on.

What this is fixing is a PHP Exception that is being raised, thus no, it is specific to this here, and no further changes are needed or planned. This is the same kind of fix as trying to load http://127.0.0.1/administrator/index.php?option=com_content&task=article.edit&id=2974628973456239845672 which is correctly handled and doesnt show a PHP exception (Error 0)

avatar PhilETaylor
PhilETaylor - comment - 6 Mar 2022

Attempt to assign property xmldata on bool is the PHP exception.

avatar brianteeman
brianteeman - comment - 6 Mar 2022

sorry - wrong site. ignore my comments (will delete) - its late.

avatar PhilETaylor
PhilETaylor - comment - 6 Mar 2022

Maybe I confused you by having the debug on in one of the screenshots... I'll replace the screenshot...

avatar PhilETaylor PhilETaylor - change - 6 Mar 2022
The description was changed
avatar PhilETaylor PhilETaylor - edited - 6 Mar 2022
avatar PhilETaylor
PhilETaylor - comment - 6 Mar 2022

I'll replace the screenshot...

Done.

avatar toivo
toivo - comment - 7 Mar 2022

I have tested this item successfully on 6f0ed71

Tested successfully in 4.1.1-dev of 6 March using Wampserver 3.2.7 and PHP 8.0.15


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

avatar toivo toivo - test_item - 7 Mar 2022 - Tested successfully
avatar PhilETaylor PhilETaylor - change - 7 Mar 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-03-07 18:34:26
Closed_By PhilETaylor
Labels Added: ?
avatar PhilETaylor PhilETaylor - close - 7 Mar 2022

Add a Comment

Login with GitHub to post a comment