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.0] Remove use of ReflectionParameter::getClass() #30581

Merged
merged 3 commits into from
Sep 23, 2020
Merged

[4.0] Remove use of ReflectionParameter::getClass() #30581

merged 3 commits into from
Sep 23, 2020

Conversation

SharkyKZ
Copy link
Contributor

@SharkyKZ SharkyKZ commented Sep 6, 2020

Summary of Changes

Remove use of ReflectionParameter::getClass() method which is deprecated in PHP 8.0.

Testing Instructions

Run Joomla on PHP 8.

Actual result BEFORE applying this Pull Request

Deprecation messages:

Deprecated: Method ReflectionParameter::getClass() is deprecated in C:\wamp\www\joomla-cms\libraries\src\Plugin\CMSPlugin.php on line 233

Expected result AFTER applying this Pull Request

No deprecation messages and plugins continue to work.

Documentation Changes Required

No.

@alikon
Copy link
Contributor

alikon commented Sep 10, 2020

I have tested this item ✅ successfully on 5e582b2

PHP Version 8.0.0-dev


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

@jwaisner
Copy link
Member

I have tested this item ✅ successfully on 5e582b2


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

@jwaisner
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 22, 2020
@wilsonge wilsonge merged commit 319edca into joomla:4.0-dev Sep 23, 2020
@wilsonge
Copy link
Contributor

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 23, 2020
@wilsonge wilsonge added this to the Joomla 4.0 milestone Sep 23, 2020
@SharkyKZ SharkyKZ deleted the j4/php8/reflectionparameter-getclass branch September 23, 2020 11:22
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Sep 26, 2020
* Joomla/4.0-dev: (844 commits)
  [4.0] Template layout select (joomla#30772)
  [4.0][CLI] com_finder use console command (joomla#30768)
  [4.0] Modifying com_actionlogs string (joomla#30758)
  [4.0] Fancy selectbox fix (joomla#30739)
  [4.0] Add missing Table Caption (joomla#30763)
  [4.0] Wrap all buttons in btn-group to improve styling (joomla#30761)
  [4.0] Cassiopeia missing string (joomla#30765)
  Improve batch text (joomla#28447)
  Fix icons not displaying (joomla#30749)
  Remove the chrome "cardGrey". The same effect can be achieved by using the module class "card-grey" with the "card" chrome (joomla#30734)
  Remove obsolete html code (joomla#30737)
  [4.0] Check out improvements related to nullable columns (joomla#30747)
  Removing card.scss overrides as we don't use those classes
  Use renderField() method to render fields (joomla#30738)
  [4.0] spelling (joomla#30742)
  [4.0] Remove use of ReflectionParameter::getClass() (joomla#30581)
  [4.0] Fix saving images (joomla#30730)
  [4.0] Cleanup cassiopeia chromes and rename "default" to "card" (joomla#30729)
  Fix hiddenLabel and add hiddenLegend form attribute (joomla#29710)
  [4.0] Fix Debug plugin to display query parameters (joomla#30717)
  ...
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
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.

5 participants