Fixes/tye-hints (#5449)

- added type-hints for DOMDocument, DOMXPath and array
- added missing documentation about optional parameter
- `if ($foo['bar'])` is not a good choice, better use
  `if (!empty($foo['bar']))` instead

There is no LOGGER_WARNING (triggering E_NOTICE about absent constant). Either (#5448)

declare it and push all other numbers higher or use LOGGER_NORMAL.

Fixes: (#5447)
- rewrote to first check if PID file is readable, then if found load it
- next addition is to cast it's content to int. Any non-integer content will
  result in 0 (as I know).
- Give warnings a voice! (xdebug's scream option showed a warning)
- added some spaces for better readability

Added a for-loop (#5442)

* Using json_decode instead of reinventing hot water

Guys, you could have told me ;)

* changed spaces to tabs

* Guess I'll need a beer ;)

Worker-details: Using json_decode instead of reinventing hot water (#5441)

* Using json_decode instead of reinventing hot water

Guys, you could have told me ;)

* changed spaces to tabs

DBStructure enhancements (#5437)

* Adding DBStructure enhancements

- Added DBStructure::rename()
- Added DBStructure::existTable()
- Added DBStructure::existColumn()

(cherry picked from commit 4ae06ec)

* Adding `pre_update_1279` method

- Added DBStructure::rename()
- Added DBStructure::existTable()
- Added DBStructure::existColumn()

(cherry picked from commit 8496d84)

* code standards

(cherry picked from commit 551d09b)

* simplify to `empty` instead `is_null`

(cherry picked from commit ce68835)

Make frio more consistent by replacing textual links with icons everywhere. (#5415)

* Make frio more consistent by replacing textual links with icons everywhere I could find them.

* [frio] Move profile add link to icon.

Merge pull request #5435 from annando/htaccess

Automatically create a .htaccess when not existing

Automatically create a .htaccess when not existing

Merge pull request #5433 from annando/mysqli

Forgotten "use"

Forgotten "use"

One more notice (#5429)

* Some more notices

* And some more notice in the worker

renamed .htaccess to .htaccess-dist and ignored .htaccess afterwards (#5428)

* renamed .htaccess to .htaccess-dist and ignored .htaccess afterwards. This
allows local changes to it without having a dirty checkout directory.

Signed-off-by: Roland Häder <roland@mxchange.org>
* Some fixes:
- removed non-neutral words
- added examples for Linux/Windows

* Better documentation and added note about **not** renaming the file.

Signed-off-by: Roland Häder <roland@mxchange.org>

We now have a pre update script (#5425)

* We now have a pre update script

* Some optimized stuff

[frio] Restyle album edit header. (#5426)

[frio] Restyle photo upload (#5427)

* Better wording in photo upload.

* [frio] Restyle photo uploader to conform to frio style.

Some more notices (#5424)

Merge pull request #5422 from MrPetovan/task/5410-rename-dba-to-DBA

Rename Friendica\Database\dba to Friendica\Database\DBA

Rename Friendica\Database\dba to Friendica\Database\DBA

Merge pull request #5420 from annando/notices

Just some more fixed notice

Merge pull request #5421 from annando/fatal

Wrong "use" for config

Wrong "use" for config

Just some more fixed notice

Merge pull request #5419 from annando/fixes

Makes Diaspora working again

Makes Diaspora working again

Merge pull request #5418 from MrPetovan/task/5410-move-dba-to-src

Move \dba to \Friendica\Database\dba

Add missing PDO* use statements in Database\dba

Remove class \dba from include/dba

Update use statement lists with new Friendica\Database\dba class

- Remove unused use statement
- Remove superfluous use statements (classes in the same namespace)
- Add missing use statements

Add new Friendica\Database\dba class

Misc cleanups (#5417)

* Fixed http://github to https://github

Signed-off-by: Roland Häder <roland@mxchange.org>
* reverted back

Signed-off-by: Roland Häder <roland@mxchange.org>
* Added type-hint 'array' as the method uses foreach() on it.

Signed-off-by: Roland Häder <roland@mxchange.org>
* Tpzo fixed ...

Signed-off-by: Roland Häder <roland@mxchange.org>
* Added missing new-line at end of file, will make some POSIX parsers being lesser
confused ...

Signed-off-by: Roland Häder <roland@mxchange.org>

Little fixes (#5414)

* We now store the item delivery data in a separate table

* Reorganized table structure

* Fix  a notice when using the profiler / ensure that permissions are strings

Small cleanup (#5416)

- added internal TODO to decide about is_result() usage
- removed semicolon (not needed here) from SQL query
- added empty line

Signed-off-by: Roland Häder <roland@mxchange.org>

Fix photo-upload and croping. (#5412)

We now store the item delivery data in a separate table (#5413)

* We now store the item delivery data in a separate table

* Reorganized table structure

Issue 5407: Workers didn't start anymore (#5409)

Workerqueue param reformat (#5403)

* Prettified the parameter layout of workerqueue

* Updated to code standards :)

* Where did I go wrong?

* Seems like someone needs to sleep ;)

New item field "Post-type" and new table "permissionset" (#5408)

* "post-type" replaces "bookmark" and "type"

* Removed some more type

* Added index to permission set

* The permission set is now stored

* The permission set is now removed upon expiry

* Post update now stores the permission set

* New file

* Permissions are now sorted

* Changed documentation

Fixes applied: (#5399)

- $_SESSION['remember_user'] did only exist here and no where else
- it seems to be accient old-lost code, so I fixed it to $a->user['nickname']
  instead
- used multi-line comment for multiple lines of on-line comments

Signed-off-by: Roland Häder <roland@mxchange.org>

Fixes: (#5404)
- fixed E_NOTICE in mod/follow.php
- fixed 2 E_NOTICE in src/Protocol/Diaspora.php
- added more type-hints for `array` type where known

Signed-off-by: Roland Häder <roland@mxchange.org>

Merge pull request #5406 from tobiasd/20180717-msg

regen master messages.po

regen master messages.po

Added warning for special characters in docs and template (#5402)

According to @annando and @MrPetovan, $emailcc is no `item` field (column) and (#5401)

should be initialized empty (I did it above the `if()` block).

Signed-off-by: Roland Häder <roland@mxchange.org>

Fix tests (#5400)

* Avoid argc side-effects

* Fix undefined array keys in JITConfigAdapter::set()

* Avoid argv side effects in ApiTest

Prettified the parameter layout of workerqueue (#5397)

* Prettified the parameter layout of workerqueue

* Updated to code standards :)

* Where did I go wrong?

Fixed E_NOTICE when no 'term' was provided (#5391)

* Fixed E_NOTICE when no 'term' was provided, maybe better encapsulate this whole
$_GET, $_POST and $_SESSION thing into a `Request` class?

Signed-off-by: Roland Häder <roland@mxchange.org>
* Rewrote to recommendation by @MrPetovan which results in really nicer code.

Signed-off-by: Roland Häder <roland@mxchange.org>

Fix silent exit in ApiTest::testApiMediaUploadWithMedia() (#5398)

switch the Vagrant VM to the new config system (#5396)

simplifying unittests (#5395)

* simplifying unittests

- use @requires instead class_exist
- define @small and @medium

* simplifying unittests

- removed unnecessary throwings

Merge pull request #5394 from MrPetovan/bug/fix-config-set-return-value

Change (P)Config::set return value to bool

Merge pull request #5295 from MrPetovan/task/4889-move-config-to-config

Move configuration to config/

Change (P)Config::set return value to bool

Fix notice in Util\Network

Fixes for E_NOTICE in mod/item.php (#5393)

* Fixes applied:
- `if (is_array($some_array))` is better code style than `if ($some_array)` as
  the `if()` block really needs an array to be found
- same with `if ($some_id)`, maybe `if ($some_id > 0)` is a more proper check
- added missing field 'emailcc' to Item::ITEM_FIELDLIST

Signed-off-by: Roland Häder <roland@mxchange.org>
* Fixes E_NOTICE of missing variable (PHP's lazyness again).

Signed-off-by: Roland Häder <roland@mxchange.org>
* Use !empty() to avoid accessing empty array.

Signed-off-by: Roland Häder <roland@mxchange.org>

Fixes for OEmbed class (#5392)

* Fixes:
- stdClass is clearly an object of `\stdClass`, so let's import it and set it
  as type-hint here
- if $o->type is not found, don't continue

Signed-off-by: Roland Häder <roland@mxchange.org>
* Fix for a fix due to 02:51 a.m. (!) edits ...

Signed-off-by: Roland Häder <roland@mxchange.org>

Improve Console/Config display for array values

- Prevent setting a config key that is an array

Add support for legacy $lang config in App->loadConfig

Remove exception for legacy config file in App->loadDatabase

Fix config value type in MemcachedCacheDriver

- Remove default config values set in CacheDriverFactory

Improve help/Config page

- Remove obsolete file-only config list
- Fix heading level
- Add transition instruction for $lang
- Add reference to config.ini.php, settings.ini.php and addon.ini.php
- Add link to help page in admin warning about legacy config file

Fixed E_NOTICE when 'title' is absent (uninitialized array key) (#5390)

Signed-off-by: Roland Häder <roland@mxchange.org>

Separate default config values between file-only and admin settings

Enforce systen.register_policy value type

Code standards in App

Fix typo in mod/admin

Remove remaining references to Config::get in include/dba

Improve console/config display

- Add visual distinction between get/set value

Fix undefined index in mod/settings

Move the addon-specific config in config/addon.ini.php

- Add sample addon.ini.php file
- Add config/addon.ini.php file to git ignore

Fix missing use in mod/ping

Fix segfault in PHP 5.6 parsing INI strings

- add documentation to generated local.ini.php file

Fix JITConfigAdapter to retrieve non-db config.* values

Fix urlpath determination on the root page

Fix undefined index notice messages in mod/install

Remove global $db variable

Replace global $a with BaseObject::getApp()

Replace global $lang with system.language

Remove deprecated functions in include/text

Fix formatting in mod/help

Chasing a segfault in PHP 5.6 tests

Remove api test pconfig debug

Fix wrong mode for App since local conf file wasn't present

Fix debug

Add yet more pconfig debug to testApiGetUserWithCustomFrioSchema

Wrong table name

Add more api test debug

Add api test debug

Put base test config in Yaml dataset

Fix Memached config value format usage

Set system.hostname in test bootstrap

Move meaningful message inside Exception thrown in Cache tests

Revert using Config class in dba

Restore system.url declaration in tests

Remove direct App instantiation from tests

Fix uses in include/dba

[tests] Prevent bin/worker.php from being called during tests

Replace direct accesses to App->config by Config::get/set calls

Add missing default config keys

- Add config.admin_nickname
- Add config.info
- Add featurelock section

Fix dbesc as a constant notice message