Merge pull request #11662 from MrPetovan/bug/11661-splitattachment-null

Ensure parameter 4 of Post\Media::splitAttachment is a boolean value

Merge pull request #11660 from Quix0r/fixes/more-type-hints-003

More type-hints - Batch 003

Fixed:
- prevent NULL from being handled over to XML::escape()
- still I wonder that an object can be? It is a string-only accepting method

Fixed indenting

Merge pull request #11663 from Quix0r/composer/upgrade

Updated composer.phar

Changes:
- added type-hints
- added some documentation
- marked some generic methods to be moved to Util\Strings class instead

Updated composer.phar

Changes:
- renamed ItemArrayFromMail() to getItemArrayFromMail() to follow naming-convetion
- added missing type-hints
- added missing documentation

Added documentation

Changes:
- as @MrPetovan pointed out, $actor can be NULL earlier and used later as NULL
- added some missing type-hints
- added missing documentation
- the added @TODO points out to avoid true|false|null for a boolean

Ensure parameter 4 of Post\Media::splitAttachment is a boolean value

Changes:
- null was 2nd argument's value before, an empty string is basically the same here

Changes:
- let's start throwing exceptions on e.g. invalid arguments instead of returning
  FALSE

Changes:
- fixed a null value handled over to Friendica\Model\APContact::getByURL()
- added missing type-hints

Changes:
- dumped default value for $input
- added unknown 'platform' which prevents an "Undefined index: platform in
  /var/www/.../src/Model/GServer.php on line 940" error

Fixes:
- Strings::isHex() should not be misused for checking on NULL

Changed:
- DBA::exists() should only be used for checking if records exists.
- if you want to check if a table exists, please ALWAYS use
  DBStructure::existsTable() instead

Changes:
- some methods now need to return bool to be compatible
- added some missing type-hints

Both declarations must be the same

Merge pull request #11659 from MrPetovan/bug/11511-relocate-quote-identifier

Add missing identifier quote in Database->replaceInTableFields

Changed:
- cannot have type-hints :-(

Ops, wrong type-hint

Changes:
- added type-hints
- added some missing documentation

Changes:
- added some documentation
- added type-hints

Changes:
- added type-hints
- added missing documentation

Changes:
- added more type-hints

Merge pull request #11657 from Quix0r/fixes/post-media-size-bigint

Size "mediumint" wasn't enough

Add missing identifier quote in Database->replaceInTableFields

- This caused hyphenated table names to fail the replace query

Changes:
- added type-hints
- changed some double-quotes to single

Changes:
 - `url` to `varbinary(1024)`
 - `preview` to `varbinary(512)`

as the previous once (half length) were not enough in real-life scenarios

Incremented again

Fixed documentation/SQL dump with ./bin/console dbstructure dump

Size "mediumint" wasn't enough:
https://digitalcourage.video/static/webseed/5a039eb2-9fbc-441d-a772-59ae3ee65c15-1080.mp4 has 6758472669 Bytes

So let's go REALLY big.

Merge pull request #11655 from Quix0r/fixes/more-type-hints-002

More type-hints and documentation added

Incremented database version

Ops, not here wanted. :-(

Changes:
- added type-hints
- added returned type-hints in interface (I checked all)

Merge pull request #11658 from tobiasd/20220619-accesskey

Accesskey was used twice

Some calls saved

Changes:
- added missing type-hints
- added documentation for a method

Changes:
- MySQL index on BLOB/TEXT can only be partial (e.g.: `column`(length))

Contact::getAccountType()'s parameter is never a string, ops

Changes:
- dbstructure(null) is no longer possible, an empty string does it

updated the Accesskey documentation accordingly

Accesskey was used twice

The accesskey on the User Settings pages for the _account settings_ and the _two factor authentication_ were the same. This PR is assinging a new one to the 2FA.

Wrong returned type, has to be array

Changes:
- added more type-hints
- DBStructure::existsTable() does no longer need array support for table name
  because this is no longer used (good work!)

Changes:
- changed to proper "use Foo\Bar;"
- added doctag
- added return type as this is fixed

Ops, also this!

Changed to suggestings (back to original) + fixed typo in scalar type

Changed back to suggestions by @MrPetovan

Naming-convention:
- variables should start lower-case: $image

Changes:
- added type-hints
- added documentation
- changed double-quotes to single

Proper type is string

Maybe fix for:

"Argument 1 passed to Friendica\Model\ItemURI::getIdByURI() must be of the type string, null given, called in Processor.php line 1219"

Changes:
- added more type-hints
- added missing documentation

Nodeinfo::getOrganization() doesn't need configuration object being inserted
when you have DI::config() around.

Added missing type-hints

Renamed variable, no need for "orig_" prefix

Continued:
- changed some double-quotes to single
- added missing type-hints
- added missing documentation
- fixed indenting a bit

Continued:
- added more type-hints
- added some missing documentation
- Return the result from DBA::delete() to let other methods know about it

Changes:
- added more type-hints
- added missing documentation

Changes:
- added missing type-hints
- added missing documentation

Merge pull request #11653 from Quix0r/fixes/more-type-hints

More type-hints added

Merge pull request #11654 from Quix0r/fixes/post-media-url-size

post-media has too small sizes for video files

Updated documentation and SQL dump, according to woodpecker tests

Continued:
- prevents a "Return value of Friendica\Core\Worker::workerProcess() must be of the type array, bool returned"

Sizes increased for:
- video files that have sizes var being int allowes
- URLs that might be longer (e.g. magnet: URLs) than varbinary(511) allows

Ops, needs to be a variable:

Error: "Cannot pass parameter 2 by reference" at /var/www/.../src/Module/DFRN/Poll.php line 36

Fixed TypeError: "Argument 1 passed to Friendica\Core\System::httpExit() must be
of the type string, null given, called in /var/www/.../src/Module/DFRN/Poll.php
on line 37"

Set type-hint for parameter $data to SimpleXMLElement as $fields in dispatch()
is the same and being handled over.

Some outside code relies on returned "false"

Fixed indenting

Changes:
- added some type-hints
- replaced most double-quotes (only Diaspora.php, later more) with single
- added some documentation
- normalized indenting in Diaspora.php (I hope I got all?)

Fixed incompatible types

Ops, wrong type again

Ops, wrong type

Continued:
- changed back to 'return false;' as other methods heavily rely on false instead
  of an empty array as pointed out by @heluecht@pirati.ca
- $fetched_contact should be initialized as an empty array, let's not make this
  code more crazier than it already is (see APContact::getByURL())

Some more type-hints added

Nore fixes

Possible fix for

Uncaught Exception TypeError: "Return value of
Friendica\Model\APContact::getByURL() must be of the type array, bool returned"

Fix for "Uncaught Exception TypeError: "Argument 1 passed to
Friendica\Model\APContact::unarchiveInbox() must be of the type string, null
given" error message

Continued:
- added more type-hints
- some methods in Diaspora returned void but integer was documented so I
  changed it to -1 to have a proper type-hint

Also need to declare $profile or otherwise an invocation of
Receiver::getReceiverForActor() will fail.

Worker::getWaitingJobForPID() can also return FALSE on failure ... :-(

Ops, wrong type-hint here, must be string ($nickname can never be an integer).

Continued:
- added type-hints
- removed out-dated documentation
- added some missing documentation

Continued:
- added more type-hints
- added some documentation
- Contact::getAccountType() should only process string, not null

Fix for non-existing record system.mobile_theme in config table

Fixed:
- $object_data['actor'] can be null, but Receiver::getReceivers()'s 2nd
  parameter expect it to be string

Some fixes:
- $gsid's default value cannot sadly be 0, it now must be null to allow some
  code work
- added some more type-hints
- documented a bit more

Added more type-hints

Added again more type-hints

Continued:
- added more type-hints
- added some missing documentation

Fixed "Argument 4 passed to Friendica\Protocol\DFRN::processVerbs() must be of the type bool"

MrPetovan brought the right one up:
> Since this is depending on remote systems, the log can quickly fill with unactionable messages.

Ops, bad type-hint here

Another incompatible method declaration fixed + type-hints added

Continued:
- more type-hints
- fixed incompatible method declarations

Added more type-hints for "App" classes