? ? Pending

User tests: Successful: Unsuccessful:

avatar izharaazmi
izharaazmi
5 Apr 2017

Pull Request for Issue #15079 .

Summary of Changes

The Hathor administrator template overrides are not maintained since long and as a consequence it now appears completely broken in J3.7. Attempts should have been made to fix the issues but its a long list to do. We are not even sure if even someone is still using it.

Therefore, I propose to remove this from Joomla as it is not in a usable state anyway.

However, if we can still fix it I am all in to my part of contribution and help it stay here until we have 4.0 at least.

Delete files is not implemented in update script to not remove Hathor during upgrades. However, I think it would still be unusable. A post-install message should work as suggested in the RFC.

Testing Instructions

Install and test that everything works fine. No traces of Hathor should be visible anywhere, especially in Template Manager and Extensions Manager.

avatar izharaazmi izharaazmi - open - 5 Apr 2017
avatar izharaazmi izharaazmi - change - 5 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Apr 2017
Category Administration com_admin Language & Strings Templates (admin)
avatar laoneo
laoneo - comment - 5 Apr 2017

I trust you that all the files do get removed with this pr. But is there something planed to inform the web site owner? We must do that properly, just removing the template is not an option IMO.

avatar mbabker
mbabker - comment - 5 Apr 2017

As is, this won't remove the template from active sites. Only new installs.

avatar brianteeman
brianteeman - comment - 5 Apr 2017

Which would actually be worse for existing users of hathor as the version in staging right now has several fixes

avatar rdeutz
rdeutz - comment - 5 Apr 2017

I checked what we can do and besides setting the default template to isis so that we don't break people sites and add a postinstall message we can't do so much. We could disable the harthor template on top (ahh close to ok) or remove the files entirely (are we looking for bad comments then ok).

avatar Bakual
Bakual - comment - 5 Apr 2017

so that we don't break people sites

Actually, nothing breaks if you remove the files. It would fall back to the Isis template and show an error.

One argument against removing the files would be so people could save existing override files. Not sure if that is a possible issue with Hathor (doubt it honestly).
We could do the file removal within the postinstall message (using the action button would trigger uninstalling the template or directly remove the files).

avatar infograf768
infograf768 - comment - 11 Apr 2017

I guess we can close this now as Hathor would remain in 3.7.0 with a postinstall message:
#15210

avatar izharaazmi
izharaazmi - comment - 11 Apr 2017

What that message is supposed to say?

avatar zero-24
zero-24 - comment - 11 Apr 2017

What that message is supposed to say?

I'm not sure what you message is supposed to say ;) There was a new PLT leadership decision that we keep hathor for 3.x (as it is allready removed from 4.0) but told any hathor users to not use this templates as it does not got any more support. So this PR "removing the template in 3.7" can be closed ;)

avatar rdeutz rdeutz - change - 11 Apr 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-04-11 10:57:48
Closed_By rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 11 Apr 2017
avatar rdeutz
rdeutz - comment - 11 Apr 2017

as @zero-24 said we will keep it till 4.0

avatar izharaazmi
izharaazmi - comment - 11 Apr 2017

Well, that's fine. But please do not forget to tell the users that "current
features may also be broken" along with "...it does not got any more
support...". Or are we planning to fix those somehow?

avatar rdeutz
rdeutz - comment - 11 Apr 2017

Or are we planning to fix those somehow?

If someone is bored, maybe ;-)

avatar zero-24
zero-24 - comment - 11 Apr 2017

Current message is: TPL_HATHOR_MESSAGE_POSTINSTALL_BODY="Currently the Hathor administrator template style is set as your site default administrator template. Please note - any new features for Joomla will only be available with the Isis template. We recommend that you switch your default backend template style to Isis. You can do this by selecting the button below. This will only change the default site setting for the administrator template. It does not change the frontend template or any individual user settings.

It may get changed but this is the current planed message we want to share ;)

avatar izharaazmi
izharaazmi - comment - 11 Apr 2017

I suggest following actions to be added:

[Switch to Isis] [Switch to Isis and remove Hathor] [Hide this Message]

avatar zero-24
zero-24 - comment - 11 Apr 2017

Please check the PR mention by JM ? We can't remove hathor as it comes back (or atlease the changed files) with the next core update ;)

avatar izharaazmi
izharaazmi - comment - 11 Apr 2017

as it comes back (or atlease the changed files) with the next core update
;)

If we are not going to be supported from now on, less chances for them to
come back :D

avatar zero-24
zero-24 - comment - 11 Apr 2017

Copyright Updates :P As well as we currently need to run the update with the Stable-Update_Package.zip wich IIRC includes any changes since 3.0 (I'm not 100% sure)

avatar izharaazmi
izharaazmi - comment - 11 Apr 2017

Sorry! Stupid me! ?

Add a Comment

Login with GitHub to post a comment