? bug ? PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
11 Mar 2023

Summary of Changes

Fix undefined key in HTMLDocument setBuffer

Testing Instructions

Apply patch.
Enable error reaporting.
Add to index.php of any template:

$this->setBuffer('<h1>FooBar</h1>', ['type' => 'component']);

Navigate to any page with this template.

Actual result BEFORE applying this Pull Request

Warning message: accesing undefinend key.

Expected result AFTER applying this Pull Request

No warning, the page render "Foobar" heading.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar joomla-cms-bot joomla-cms-bot - change - 11 Mar 2023
Category Libraries
avatar Fedik Fedik - open - 11 Mar 2023
avatar Fedik Fedik - change - 11 Mar 2023
Status New Pending
avatar Quy
Quy - comment - 12 Mar 2023

No warnings however in the browser's console.

40081

avatar Fedik
Fedik - comment - 12 Mar 2023

Yes, that is expected, because the test instruction overrides whole component output, and every srcipt will be broken.
But that only for testing setBuffer method, so all good ?

avatar toivo toivo - test_item - 17 Mar 2023 - Tested successfully
avatar toivo
toivo - comment - 17 Mar 2023

I have tested this item successfully on 17e7a66

Tested successfully in Joomla 4.3.10-dev of 16 March in PHP 8.1.10


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

avatar viocassel viocassel - test_item - 17 Mar 2023 - Tested successfully
avatar viocassel
viocassel - comment - 17 Mar 2023

I have tested this item successfully on 17e7a66


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

avatar alikon alikon - change - 17 Mar 2023
The description was changed
Status Pending Ready to Commit
avatar joomla-cms-bot joomla-cms-bot - edited - 17 Mar 2023
avatar alikon
alikon - comment - 17 Mar 2023

RTC


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

avatar HLeithner
HLeithner - comment - 2 May 2023

This pull request has been automatically rebased to 4.3-dev.

avatar obuisard obuisard - change - 5 May 2023
Labels Added: ? bug ? ?
avatar obuisard obuisard - change - 5 May 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-05 19:57:09
Closed_By obuisard
Labels Added: PR-4.3-dev
avatar obuisard obuisard - close - 5 May 2023
avatar obuisard obuisard - merge - 5 May 2023
avatar obuisard
obuisard - comment - 5 May 2023

Thank you Fedir @Fedik :-)

Add a Comment

Login with GitHub to post a comment