Avoid no needed call on parse2str() when there is nothing to parse (#5476)

Merge pull request #5439 from MrPetovan/task/5410-remove-dbesc

Move DBA to src/ part 4.1: Remove dbesc()

Added true to json_decode (#5471)

Remove dbesc() and dba_timer() function declarations

Rename dbesc to DBA::escape

Update DBA::escape to match dbesc()

Merge pull request #5438 from MrPetovan/task/5410-remove-dbm-class

Move DBA to src/ part 3.2: Remove DBM class

Delete Database\DBM class file

Rename DBA::esc_array to DBA::escapeArray

- Rename internal method DBA::esc_array_callback to
DBA::escapeArrayCallback

Rename DBA::is_result to DBA::isResult

Rename DBM method calls to DBA method calls

Use DateTimeFormat::utc() instead of DBM::date()

Add Database\DBM methods to Database\DBA

Add missing space between PHP path and script in App->proc_run (#5468)

Merge pull request #5431 from MrPetovan/task/5410-rename-database-methods

Move DBA to src/ part 3.1: Rename DBA methods

Fix Fatal errors in Protocol\OStatus (#5466)

And some more removed notices in the core (#5465)

Using getopt for CLI arguments (#5446)

* Adding Argument class to Friendica

* Adding Argument class to Friendica

* Adding Argument class to Friendica

* fixing arguments for `spawnWorker`

* Adding `use Friendica\BaseObject` to `ApiTest.php`

* Refactoring the argument-usages of Friendica

* Refactoring the argument-usages of Friendica

* removing superfluous []

Cleaned up: (#5461)

- no more x() (deprecated) usage but empty() instead
- fixed mixing of space/tab indending
- merged else/if block goether in elseif() (lesser nested code blocks)

Cleanup: (#5463)

- rewrote x() to empty() as it is deprecated
- fixed indending for some entries

Fixes: (#5462)
- fixed E_NOTICE about missing array element "page-flags-raw" (should be
  "page_flags_raw"?)
- same with account-type-raw -> account_type_raw
- added spaces to nicely indent code

Prevents 4 E_NOTICEs when POST fields are not set. (#5458)

* Prevents 4 E_NOTICEs when POST fields are not set.

* Don't use x().

* Replaced usage of x() which is deprecated with !isset().

* Use empty() according to MrPetovan

Rename DBA::get_db to DBA::getConnection

- Rename DBA::$db to DBA::$connection

Rename DBA::num_rows to DBA::numRows

Rename DBA::server_info to DBA::serverInfo

Rename DBA::inArray to DBA::toArray

Rename DBA::fetch_first to DBA::fetchFirst

Rename DBA::database_name to DBA::databaseName

Rename DBA::clean_query to DBA::cleanQuery

Rename DBA::any_value_fallback to DBA::anyValueFallback

Rename DBA::affected_rows to DBA::affectedRows

Need to load "verb" from notify table, too. This whole thing is a maintenance (#5459)

nightmare. Let's use the new database frontend/backend system and load all
columns anywhere the "entity" is requested.

Fix Fatal error in Protocol\Diaspora (#5460)

Fixed E_NOTICE when no valid result has been returned. (#5457)

Fixed E_NOTICEs when uploading profile picture (#5456)

* Fixed E_NOTICEs when uploading profile picture.

* Fixed E_NOTICEs when $_POST doesn't contain these fields.

Uncommon logger levels in Friendica (#5453)

* "normal" is an uncommon logger level:
- changed LOGGER_NORMAL -> LOGGER_INFO
- added LOGGER_WARNING (a common logger level)

* Used constants instead of values (MrPetovan)

Fix yet another Fatal Error in Protocol\OStatus (#5454)

 Fix another Fatal error in Protocol\OStatus (#5451)

* Fix Fatal error in Protocol\OStatus

* Fix another Fatal error in Protocol\OStatus

Fixed E_NOTICE in boot.php and DBA class (#5430)

* Fixes:
- fixed missing variable $port (MySQL: 3306)
- "imported" mysqli class

Signed-off-by: Roland Häder <roland@mxchange.org>
* Fixed:
- better use `false` and `$port > 0`

* And better only provide `$port` when larger zero.

* Initialize `$port` with zero value (int) and not `false` (bool).

* Removed duplicate mysqli "import".

* `$post_update` is no longer used. Instead `$prefix` needs to be checked.

Fix Fatal error in Protocol\OStatus (#5450)

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