Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4] add missing import for exception #31081

Merged
merged 1 commit into from
Oct 15, 2020
Merged

[4] add missing import for exception #31081

merged 1 commit into from
Oct 15, 2020

Conversation

PhilETaylor
Copy link
Contributor

Closes #31063

Summary of Changes

Add missing import so that Exception\ResourceNotFound is correctly resolved

Testing Instructions

Code Review

Signed-off-by: Phil E. Taylor <[email protected]>
@ChristineWk
Copy link

I have tested this item ✅ successfully on 5089f4b


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

@ceford
Copy link
Contributor

ceford commented Oct 14, 2020

I have tested this item ✅ successfully on 5089f4b

Confirmed code change use matches call.


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

@Quy Quy removed the PR-4.0-dev label Oct 14, 2020
@Quy
Copy link
Contributor

Quy commented Oct 14, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 14, 2020
@Quy Quy added the PR-4.0-dev label Oct 14, 2020
@richard67 richard67 merged commit 59732cb into joomla:4.0-dev Oct 15, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Oct 15, 2020
@richard67
Copy link
Member

Thanks!

@richard67 richard67 added this to the Joomla 4.0 milestone Oct 15, 2020
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Add missing import so that "Exception\ResourceNotFound" is correctly resolved.
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