? ? ? Pending

User tests: Successful: 2 alikon, franz-wohlkoenig Unsuccessful: 0

avatar infograf768
infograf768
15 Mar 2017

Pull Request for discussion in #14623

Summary of Changes

Hathor will now use the Global language string already used in Isis for its Associations slider

Descriptions for these fieldsets were no more used for a while. Modified code.

Related language strings are marked as deprecated

Testing Instructions

Set a multilingual site with associations.
Set Hathor as your default template.

Edit or create an article
Edit or create a category
Edit or create a contact
Edit or create a Newsfeed
Edit or create a menu item

Expected result

before patch one will get

hathor_before

After patch

hathor after

The other changes in code just have to be reviewed.

avatar infograf768 infograf768 - open - 15 Mar 2017
avatar infograf768 infograf768 - change - 15 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Mar 2017
Category Administration com_categories com_contact com_content com_menus com_newsfeeds Language & Strings Templates (admin)
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Mar 2017

I have tested this item successfully on d20aff2

In all 5 Types Slider-Name is Associations.


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

avatar infograf768
infograf768 - comment - 15 Mar 2017

@brianteeman
Please test.

avatar alikon
alikon - comment - 15 Mar 2017

please kill hator asap

avatar alikon alikon - test_item - 15 Mar 2017 - Tested successfully
avatar alikon
alikon - comment - 15 Mar 2017

I have tested this item successfully on d20aff2


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Mar 2017

@alikon read today "not before J4".

avatar alikon
alikon - comment - 15 Mar 2017

is for lack of PR or for b/c ?

avatar brianteeman
brianteeman - comment - 15 Mar 2017

b/c - sadly

PR looks ok to me on review

http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

On 15 March 2017 at 17:43, Nicola Galgano notifications@github.com wrote:

is for lack of PR or for b/c ?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#14633 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8bmmmDsKJDXNbFIocDGd2SRfGFnDks5rmCNGgaJpZM4Mdtyv
.

--
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar infograf768
infograf768 - comment - 15 Mar 2017

RTC. Thanks for testing.


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

avatar infograf768 infograf768 - change - 15 Mar 2017
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 15 Mar 2017

rtc


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

avatar infograf768
infograf768 - comment - 15 Mar 2017

@alikon
the pr also deprecates some lang strings an kills in code useless adding description to fieldset. i mean it does not only normalise hathor. ;)

avatar rdeutz rdeutz - change - 15 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-15 20:23:56
Closed_By rdeutz
Labels Added: ? ? ?
avatar rdeutz rdeutz - close - 15 Mar 2017
avatar rdeutz rdeutz - merge - 15 Mar 2017

Add a Comment

Login with GitHub to post a comment