Rename variable to indicate that it can be port and socket.

No need to check the port number as this will fail anyway.

Move getValue() to where required. This actually is not required "global data".

Move check for : to where it belongs, getting rid of $socket = false;

Extract common variables of $connectionParams.

Inline $port expression, getting rid of $port variable.

Fixes based on suggestions

I use the term socket for any extension, either unix socket, or internet socket (port).

I check if the socket is all digits
* only integers 0 and larger would pass this test.
I then check if the string is less than or equal to the maximum port number.

By using "if($socket)" I make sure socket isn't false, empty, or the string '0'.

I don't believe I need to initialize $port because $port will always be set if $socket is true. Please show me if I am wrong here. Thanks

Update notes on dbhost

Fix variable not always being defined.

Work with MySQL Sockets

This passes anything that is not a valid port (0<int<65535) as a unix socket.
I tested this with unix sockets; this needs to be tested with a non-standard mysql port as well but I don't foresee any issues.

To use a unix socket, even one different than PHP's mysql.default_socket..
* Database Host = localhost:/path/to/socket

Merge pull request #8882 from owncloud/remove-placeholdercall

remove obsolete placeholder call, fix #8881

Merge pull request #8724 from owncloud/update-sabre-2.0

Update SabreDAV to 1.8.10

Merge pull request #8824 from owncloud/mail_template_editing_in_webui

minimal mail template editor for administrators

remove obsolete placeholder call, fix #8881

Merge pull request #8471 from owncloud/versioning_no_longer_track_the_size

[versioning] let the file cache track the size of the versions

Merge pull request #8876 from owncloud/design-css-settings

personal settings: unbold clients header and remove obsolete CSS

use the right interface \Sabre\DAVACL\PrincipalBackend\BackendInterface

personal settings: unbold clients header and remove obsolete CSS

Merge pull request #8791 from owncloud/share-overview-sharewithstatus

Update share action text to display owner/recipients

Merge pull request #8875 from owncloud/app-css-relative

explicit position:relative for app-content, fixes first run layout

remove obsolete position rule from files app

explicit position:relative for app-content, fixes first run layout

fixes during test

Merge pull request #8822 from owncloud/cache-change-propagator

[WIP] Improved propagation of cache changes

Merge pull request #8844 from owncloud/design-icons

[WIP] simplify icons, remove shadow

Upgrade SabreDAV to 1.8.10
Updating SabreDAV namespaces

Merge pull request #8701 from owncloud/slide-up-toggle

Create reusable data attribute for apps setting slideup

Merge pull request #8853 from owncloud/design-typeface

Use Open Sans as typeface

Merge pull request #8860 from owncloud/phpstorm-code-style

add phpstorm code style

IE -> IE8 only

move from tff to woff and fix IE8

add phpstorm code style

Do not update action icon recipients for file list

Since the files app doesn't have the recipient information on load it
cannot initially render the recipients in the action icons.

For this reason the action icon will not be updated with the recipients
after changing the shares, for consistency.

Use recipient display names when updating shares in the UI

Since OC.Share didn't have any array containing the list of shares for
the current file, OC.Share.currentShares has been introduced to contain
the full share item structure instead of the reduced one
OC.Share.itemShares.

The event "sharesChanged" is now passing OC.Share.currentShares, which
itself includes the display name to be displayed for the recipients in
the action icon.

Merge pull request #8662 from owncloud/f-lock

f-lock

Fix unit tests

Merge pull request #8842 from owncloud/ldap_cli_methods

command line tools to see, modify and test existing LDAP configurations

Merge pull request #8849 from owncloud/trash_expire_fix

get files in trash from file cache

fixing typo

Merge pull request #8468 from owncloud/remove-infieldlabels

Remove infieldlabels

workaround to get the unit test going

[tx-robot] updated from transifex

get the storage from the view

Use ->instanceOfStorage() not instanceof for Storage instances.

Can't use assertInstanceOf on wrapped storage; use assertTrue(instanceOfStorage() instead.

only lock if unlink is called for a file

fix unit test after adding the additional parameter to addStorageWrapper()

name the storage wrapper to make sure that we don't apply the wrapper multiple times

don't ask for fileInfo if we already have one

Normalize lock exception messages.

make sure to close the stream at the end of each test

Restore the reference to this used exception class.

Move Lock to private namespace, add interface. Update PHPDoc.

Don't error out on console.

Removed unused vars/declarations, update PHPDoc.

Fix typo

Fix typo

Remove unused methods.

Respect locked files, surface correct exception.

Simpler log unique id.

Add actual locking and log changes necessary for debugging.

Continued flock work.

flock changes.  Work in progress.

unbold the username

Fix size calculation during recursive scan

Merge pull request #8686 from owncloud/session-early

Make the session available as early as possible

Merge pull request #8698 from owncloud/sharing_improve_permissions

get permissions directly from share storage to avoid additional db calls

do not use italic typeface style, instead lighter color

Fix casing and PHPDoc

Merge pull request #7930 from owncloud/fix_7785_master

LDAP: put out fetching of user meta data into a fully tested class of it...

use Open Sans as typeface

Fix merge conflict

Merge pull request #8264 from owncloud/app-enable-by-group

Add the option to enable apps only for specific groups

Merge pull request #8809 from owncloud/appconfig-caching

Improve caching in AppConfig

convert mail template editor to app framework for a restful api

Merge pull request #8837 from owncloud/issue/8720

Fix cursor and background of app navigation

get files in trash from file cache

Merge pull request #8840 from owncloud/share_settings

by default don't show option to send mail notifications for shares

Merge pull request #8577 from owncloud/apps-sidebar-narrower

reduce width of app sidebar a bit

Merge pull request #8843 from owncloud/design-detail-sharedropdown

fix positioning issue of share dropdown in intermediate widths

Merge pull request #7015 from owncloud/db-migration-check-copy

Check database migration on a copy of the table first

move to updated version of placeholder

simplify logout and caret icons, remove shadow

fix text alignment in input fields

fix icon alignment in input fields

add missing placeholder shim for change password fields in IE8/9

remove tilde selector for IE8 compatibility

remove scruntinizer config for infield

fix show password icon in personal settings

infield label removal: remove last occurences of infield labels

infield label removal: remove JS file and references to it

infield label removal: fix installation screen

first step of infield label removal, fix login screen

Merge pull request #8827 from owncloud/add-sqlite-info

add info about sqlite on setup page

Merge pull request #8838 from owncloud/backgroundjob-legacy-gc

Support having legacy `OC_Cache_FileGlobalGC` background jobs

Merge pull request #7151 from owncloud/user-jquery

Continuing the New User management

remove specific sidebar size from Files app

reduce width of app sidebar a bit

fix positioning issue of share dropdown in intermediate widths

get rid of @read in PHP doc after rebase