? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
5 May 2017

Pull Request for Issue #15822.

Summary of Changes

Improve tooltip to show what values should be accepted
Allow Joomla to connect correctly to Redis that is using sock and not TCP
LANGUAGE FILE CHANGE (no idea how to propagate that to translators)

Testing Instructions

Set up redis.conf with port 0 and listening on a sock
port 0
unixsocket /tmp/redis.sock
restart redis

Joomla global Config as such:
screen shot 2017-05-05 at 13 56 08

Expected result

no matter what port you enter, joomla will ignore it, as long as the sock file is correct, Joomla can connect and cache

avatar PhilETaylor PhilETaylor - open - 5 May 2017
avatar PhilETaylor PhilETaylor - change - 5 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 May 2017
Category Administration Language & Strings Libraries
avatar PhilETaylor PhilETaylor - change - 5 May 2017
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 5 May 2017

no idea how to propagate that to translators)

Don't worry about that.

avatar steelAXIS
steelAXIS - comment - 5 May 2017

Tested and working as expected on my side.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 May 2017

@steelAXIS can i alter your Test as successfully at Issue Tracker?

avatar steelAXIS
steelAXIS - comment - 5 May 2017

@franz-wohlkoenig sure go ahead.

avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 5 May 2017 - steelAXIS: Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 May 2017

done, thanks.

avatar brianteeman
brianteeman - comment - 13 Aug 2017

@PhilETaylor can you resolve the conflicts please?

avatar PhilETaylor
PhilETaylor - comment - 13 Aug 2017

@brianteeman This is not a conflict. This is 10 tonnes of Joomla progressing (libraries/joomla/cache doesnt exist any more, lots of refactoring of libraries and vendors) while a PR remains unmerged. This PR is no longer fit for use.

avatar PhilETaylor PhilETaylor - change - 13 Aug 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-08-13 20:56:15
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 13 Aug 2017
avatar mbabker
mbabker - comment - 13 Aug 2017

Phil it seriously should just be a git merge upstream staging and done. The vast majority of PRs touching moved libraries files have been easily sync'd with a merge, I've seen very few merge conflicts beyond the file rename.

(And I wrote this before realizing the new PR was there, but just for general reference...)

avatar PhilETaylor
PhilETaylor - comment - 13 Aug 2017

i was/am in bed and exhausted and too sleepy to dig :)

You cannot merge changes to a file that no longer exists though right ? This is what happened with this PR.

avatar mbabker
mbabker - comment - 13 Aug 2017

As long as the file was moved in a git rename operation then merging the upstream branch should track that and attempt to migrate your changes to the new file path.

You're screwed though if git tracks it as deleting the old file and creating a new file (git proper, not how GitHub displays things).

Add a Comment

Login with GitHub to post a comment