Skip to content

Conversation

@joomdonation
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

This PR fixes fatal error when saving template options from frontend.

Testing Instructions

  • Use Joomla 6.0 nightly build
  • Create a menu item to link to Configuration -> Display Template Options menu item type
  • Login to frontend of your site using super user accountt
  • Access to the above menu item and try to change template options, save it

Actual result BEFORE applying this Pull Request

You got fatal error

Expected result AFTER applying this Pull Request

No error, template options saved properly

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

@exlemor
Copy link

exlemor commented Mar 23, 2025

I have tested this item ✅ successfully on ccc73d1

Another quick and clear test - I was able to test this successfully.

Thanks @joomdonation.


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

@krishnaGandhi11
Copy link

I have tested this item ✅ successfully on ccc73d1

Tested Successfully! @joomdonation .


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

@QuyTon
Copy link
Contributor

QuyTon commented Mar 24, 2025

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 24, 2025
@Bodge-IT Bodge-IT merged commit 60d1f0e into joomla:6.0-dev Mar 26, 2025
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 26, 2025
@Bodge-IT Bodge-IT added this to the Joomla! 6.0.0 milestone Mar 26, 2025
@joomdonation joomdonation deleted the fix_save_template_options_error branch March 27, 2025 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants