Merge pull request #5522 from Quix0r/fixes/portable-contact-empty-body

Fixed fatal error about when returned 'body' field is empty

Fixed fatal error about when returned 'body' field is empty:

````
PHP Fatal error:  Uncaught TypeError: Argument 1 passed to Friendica\Protocol\PortableContact::discoverServer() must be of the type array, null given, called in /var/www/../src/Protocol/PortableContact.php on line 1664 and defined in /var/www/../src/Protocol/PortableContact.php:1765
Stack trace:
````

Merge pull request #5519 from annando/test-2

Moved expiring of item content

Merge pull request #5520 from annando/notices

Some more notices removed

And another notice removed

Removed some test logging

Some more notices removed

Moved expiring of item content

Trying to clean up a little bit more (#5517)

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

* The permission set is now removed upon expiry

* Post update now stores the permission set

* We don't store the permissions in the item table anymore

* Postupdate now handles "postopts" as well

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

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

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

* The permission set is now removed upon expiry

* The permission set is now removed upon expiry

* We don't store the permissions in the item table anymore

* Postupdate now handles "postopts" as well

* Multiplicated update functions

* And some more duplicated functions ...

* Removed duplicate functions

 Fix for post update  (#5513)

* "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

* The permission set is now used for item permissions

* Check for allow_cid, ... is superfluous. Checking for "private" is enough

* We query the permissionset

* Permissions are displayed correctly

* Changed index

* We don't store the permissions in the item table anymore

* Permission fields are now deprecated

* Reversed ...

* Postupdate now handles "postopts" as well

* Set deprecated fields to "null" if empty

* Postupdates are enabled again

* Fix for post update

Missing stuff in "develop"? (#5516)

* "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

* The permission set is now used for item permissions

* Check for allow_cid, ... is superfluous. Checking for "private" is enough

* We query the permissionset

* Permissions are displayed correctly

* Changed index

* We don't store the permissions in the item table anymore

* Permission fields are now deprecated

* Reversed ...

* Postupdate now handles "postopts" as well

* Set deprecated fields to "null" if empty

* Postupdates are enabled again

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

* The permission set is now stored

* The permission set is now removed upon expiry

* Postupdate now handles "postopts" as well

Fix parse error in Protocol\PortableContact (#5515)

Possible fix for #5470 - json_decode() (#5511)

* Possible fix for #5470:
- $data is not an object like stdClass but an array
- newer PHP versions doesn't allow cross-access like following:

  $object['foo'] = 123;
  $array->foo = 123;

- added type-hints for private methods for above cases
- used `if (empty($foo)) instead of just `if ($foo)` preventing some nasty
  E_NOTICE
- added some empty lines for better readability

* Rewrite:
- mixture of object/array was here, causing under newer PHP versions some E_NOTICE
- this has been now finally fixed by converting any `object` type to an
  associative `array`
- also changed `is_object()` to `is_array()`

Moved global PAGE_* to Profile class (#5500)

* Rewrites:
- moved PAGE_* to Friendica\Model\Profile class

* Fixed more rewrites from plain (global namespace) PAGE_* to Friendica\Models\Profile class

* CR request:
- moved all PAGE_* constants to Friendica\Model\Contact class
- fixed all references of both classes

* CR request:
- moved ACCOUNT_TYPE_* constants from boot.php to Contact::ACCOUNT_TYPE_*

* Just copy-pasted this code from boot.php, needs to be changed to `const ACCOUNT_TYPE_FOO = x;`

* Ops, melting brain cells here ... :-/

Postupdate is now working again (#5512)

* "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

* The permission set is now used for item permissions

* Check for allow_cid, ... is superfluous. Checking for "private" is enough

* We query the permissionset

* Permissions are displayed correctly

* Changed index

* We don't store the permissions in the item table anymore

* Permission fields are now deprecated

* Reversed ...

* Postupdate now handles "postopts" as well

* Set deprecated fields to "null" if empty

* Postupdates are enabled again

Merge pull request #5509 from friendica/revert-5506-fixes/portable-contact

Revert "Possible fix for #5470 and empty() usage"

Revert "Possible fix for #5470: (#5506)"

This reverts commit 1e20fd84f7e942c9cce8c1fbdecf6ec73b4c8a43.

Revert "Continued: (#5508)"

This reverts commit 592190892c17ab61cce132dee1a4a2020575b180.

Continued: (#5508)

- `App` is now "imported" to avoid having \Friendica\App::* everywhere

Possible fix for #5470: (#5506)

- $data is not an object like stdClass but an array
- newer PHP versions doesn't allow cross-access like following:

  $object['foo'] = 123;
  $array->foo = 123;

- added type-hints for private methods for above cases
- used `if (empty($foo)) instead of just `if ($foo)` preventing some nasty
  E_NOTICE
- added some empty lines for better readability

[frio] Fix wrongly removed content wrapper and fix CSS instead. (#5496) (#5505)

Item storage: Permissions aren't stored in the items anymore (#5495)

* The permission set is now used for item permissions

* Check for allow_cid, ... is superfluous. Checking for "private" is enough

* We query the permissionset

* Permissions are displayed correctly

* Changed index

* We don't store the permissions in the item table anymore

* Permission fields are now deprecated

* Reversed ...

[frio] Remove frame around recent photos. (#5496) (#5499)

Also layout file properly.

[frio] Fix layout of profile aside. (#5497) (#5498)

See issue (#5497)[https://github.com/friendica/friendica/issues/5497] for details.

Bugfix: Session handling hadn't been initialized (#5493)

Merge pull request #5491 from tobiasd/20180725-issue1391

Make profile settings available in the user settings

add accesskey to the docs

Make profile settings available in the user settings

Fixes/rewrites on videos.php (#5489)

* Fixes/rewrites:
- use empty()/isset() instead of deprecated x()
- merged 2 nested if() blocks into one
- avoided nested if() block inside else block by rewriting it to elseif()
- $contact_id is an integer, let's test on > 0 here
- added a lot spaces and some empty lines for better readability

* CR request:
- removed stray spaces
- added some where they improve readability

Moved CONTACT_* constants to Friendica\Model\Contact class, lesser in global namespace (#5490)

* Rewrite:
- moved all CONTACT_* constants from boot.php to Contact class

* CR request:
- renamed Contact::CONTACT_IS_* -> Contact::* ;-)

Removal of x() and added missing columns of contact (#5479)

* Fixes:
- got rid of x() in whole mod/item.php file
- added missing 'prv' and 'forum' columns, maybe one day, let's rewrite this
  again. it is kinda a maintenance nightmare, keeping up with column renames and
  such. Well, for now it should be okay.

* Should be DBA::isResult()

Fallback to Diaspora if DFRN transmission fails (#5487)

Merge pull request #5452 from MrPetovan/bug/5443-fix-worker-notices

Fix notices in OEmbed et al.

Reduce cache time for OEmbed error

[frio] Fix profile edit links. [#5474] (#5475)

Store guid and plink with event (#5485)

* Store guid and plink with event

* One space too much

Move addLanguageToItemArray call when body is ensured

Added bootstrap tables to queues (#5484)

* Added bootstrap tables to queues

* Changed to double quotes

Fix undefined oembed properties

- Add Object\OEmbed class

Fix Undefined index: nurl in Protocol\PortableContact

Fix by reference notice in Protocol\PortableContact

Fix undefined index body in Model\Item

Fix getimagesizefromstring() read errors

Some removed notices when viewing events (#5483)

Fixes for E_NOTICE in editpost.php (#5480)

* Added missing columns 'wall' and 'post-type'.

* Convention:
- curly brace after a a method/function declaration belongs to next line
- added space for better readability

Merge pull request #5482 from annando/issue-5467a

Issue 5467: Cron jobs hadn't been executed

Issue 5467: Cron jobs hadn't been executed

Merge pull request #5481 from annando/issue-5467

Issue 5467: Fix worker and daemon

Some code cleanup

Issue 5467: Fix worker and daemon

Merge pull request #5478 from MrPetovan/task/5410-add-missing-use-statements

Add missing use statements after dbesc conversion

Fixes: (#5477)
- use DBA::isResult() and not count()
- fixed 4 E_NOTICE when POST fields are not set

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

Ad missing use statements after dbesc conversion

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.