-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Make JAuthentication autoloaded, split 2 classes to 2 files #11023
Conversation
@@ -1424,6 +1424,8 @@ public function deleteUnexistingFiles() | |||
'/administrator/templates/isis/js/jquery.js', | |||
'/administrator/templates/isis/js/bootstrap.min.js', | |||
'/media/system/js/permissions.min.js', | |||
// Joomla __DEPLOY_VERSION__ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this (#10883) needs to be merged for __DEPLOY_VERSION__
to work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm counting on that being merged before this PR ever makes it to a stable release 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😉
I have tested this item ✅ successfully on 3233daa This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11023. |
I have not tested this item. This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11023. |
I have tested this item ✅ successfully on 3233daa This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11023. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11023. |
Merged with 761bf53 - thanks! |
Summary of Changes
The
JAuthentication
class requires manual import and the file that holds this class contains two classes (JAuthentication
andJAuthenticationResponse
). Move the file so it can be autoloaded and split the response class to a new file to follow the one class per file convention.Testing Instructions
User authentication still works correctly, on update the old file is deleted.