? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
7 Aug 2017

As discussed with @mbabker we can permanently resolve the issue of the name of the installation folder from being translated by removing it from the string and inserting it dynamically using JText::sprintf

All other attempts have failed so this is the only way we can ensure that users are presented with the correct information

Updated PR to work around some guthub issue with #17276

avatar joomla-cms-bot joomla-cms-bot - change - 7 Aug 2017
Category Installation Language & Strings
avatar brianteeman brianteeman - open - 7 Aug 2017
avatar brianteeman brianteeman - change - 7 Aug 2017
Status New Pending
avatar mbabker mbabker - change - 7 Aug 2017
Labels Added: ? ?
avatar mbabker
mbabker - comment - 7 Aug 2017

For whatever reason it was still missing that file's changes here, I just fixed it. We should be good now.

avatar Quy Quy - test_item - 8 Aug 2017 - Tested successfully
avatar Quy
Quy - comment - 8 Aug 2017

I have tested this item successfully on 9c985e8

Code review.


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

avatar brianteeman brianteeman - change - 11 Aug 2017
Title
Installations2
Installation
avatar brianteeman brianteeman - edited - 11 Aug 2017
avatar mbabker mbabker - change - 15 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-15 11:56:05
Closed_By mbabker
avatar mbabker mbabker - close - 15 Aug 2017
avatar mbabker mbabker - merge - 15 Aug 2017
avatar brianteeman
brianteeman - comment - 15 Aug 2017

Thanks for merging

avatar zero-24
zero-24 - comment - 15 Aug 2017

why did we merge that with a red drone: http://213.160.72.75/joomla/joomla-cms/7860 ?

image

I'm going to prepare a PR..

avatar mbabker
mbabker - comment - 15 Aug 2017

Missed that, my bad.

avatar zero-24
zero-24 - comment - 15 Aug 2017

here it is #17544

Add a Comment

Login with GitHub to post a comment