Skip to content

Fix DataFormatter::formatCallerInfo to handle Closure#38581

Merged
wilsonge merged 2 commits intojoomla:4.2-devfrom
Fedik:debug-format-caller-fix
Sep 6, 2022
Merged

Fix DataFormatter::formatCallerInfo to handle Closure#38581
wilsonge merged 2 commits intojoomla:4.2-devfrom
Fedik:debug-format-caller-fix

Conversation

@Fedik
Copy link
Copy Markdown
Member

@Fedik Fedik commented Aug 24, 2022

Pull Request for Issue #38556 .

Summary of Changes

Correct DataFormatter::formatCallerInfo to handle Closure variable

Testing Instructions

Please follow #38556

Actual result BEFORE applying this Pull Request

An error

Expected result AFTER applying this Pull Request

No error

Documentation Changes Required

nope

@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 04c3177


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

Co-authored-by: Tobias Zulauf <zero-24@users.noreply.github.com>
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 24, 2022

I have tested this item ✅ successfully on 532196e


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 24, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 24, 2022
@wilsonge wilsonge merged commit 691cc0e into joomla:4.2-dev Sep 6, 2022
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Sep 6, 2022

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 6, 2022
@wilsonge wilsonge added this to the Joomla! 4.2.3 milestone Sep 6, 2022
@Fedik Fedik deleted the debug-format-caller-fix branch September 6, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants