Merge pull request #11689 from Quix0r/rewrites/double-quotes-single

Changed more double-quotes to single

Made @TODO more clear?

Merge pull request #11688 from MrPetovan/bug/11635-mod-photos-posted

Replace obsolete posted sort field by created in mod\photos

Replace obsolete posted sort field by created in mod\photos

Add tests for BBCode::fetchShareAttributes

Centralize share tag attribute extraction code in Content\Text\BBCode

Changes:
- added type-hints
- added documentation

Changes:
- let's get rid of that `Database::exists()` is being misused for checking table existence
- reformatted array

Changed double-quotes to single

Changes:
- added some type-hints
- added documentation

Space for the rescue. ;-)

Reformatted array

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

Changed doubled-quotes to single

Merge pull request #11685 from MrPetovan/bug/11638-gserver_site_name-length

Add explicit gserver.site_name string truncate in Model\GServer

Add explicit gserver.site_name string truncate in Model\GServer

- Avoids "Data too long for column 'site_name'" database errors

Merge pull request #11686 from MrPetovan/bug/11636-emailer-message-id-check

Fix off-by-one error in Message ID header count check in Util\Emailer

Fix off-by-one error in Message ID header count check in Util\Emailer

Merge pull request #11683 from MrPetovan/bug/fatal-errors

Fix a couple Fatal Errors

Update default value for LocalRelationship->priority to avoid null values

- Address https://github.com/friendica/friendica/issues/11630#issuecomment-1166192993

Update return value type-hint of GServer::getProtocol

- Address https://github.com/friendica/friendica/issues/11630#issuecomment-1164880874

Merge pull request #11681 from Quix0r/fixes/item-guidfromuri-allow-null

Allowing Item::getFromUri()'s 2nd parameter a null value

Merge pull request #11680 from nupplaphil/feat/log_forward

Respect Forwarded-For headers

Changes:
- Made Item::guidFromUri()'s $host parameter optional (allowing null)
- added some type-hints
- added documentation
- this may fix reported issue in #11630

Move documentation :-)

Move documentation :-)

adaptions :-)

add more doc

Add copyright

Respect Forwarded-For headers

Merge pull request #11679 from Quix0r/rewrites/autotest

Rewrite of autotest.sh

Rewrite:
- rewrote autotest.sh and moved it to bin/dev/ as this is clearly a developer
  script
- the rewrite's goal is towards Shell/POSIX-compliance (e.g. ${VAR} and not $VAR)

Merge pull request #11677 from Quix0r/rewrites/type-hints-doc-001

Rewrites/type hints doc 001

Changes:
- documented null value
- two ' to much!

No need to log $tid here

Reformatted code

Changes:
- added some nodes to OpenSearch descriptor
- still no UTF-8 encoding!
- added some type-hints

Changes:
- added some type-hints
- added documentation

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

More type-hints added

Changes:
- added type-hints
- added missing documentation

Changes:
- reformatted some array
- added missing documentation
- added type-hints
- changed double-quotes to single

Added more type-hints

Added some type-hints

Added some missing type-hints

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

Throw IAE again (should never become visible)

Ops, cannot return Thread?

Changed:
- Introduced InvalidArgumentException (should never come)
- added type-hints

Continued:
- renamed varibales to $camelCase
- added type-hints
- updated documentation

Merge pull request #11678 from Quix0r/fixes/send-contact-accept-id-string

Fix for Transmitter::sendContactAccept() wrong 2nd parameter type

$id in Transmitter::sendContactAccept() is a string, see Introduction class

Merge pull request #11676 from Quix0r/fixes/array-element-missing

Maybe this fixes missing array element

Merge pull request #11675 from Quix0r/features/smarty-sub-directories

Enabled sub-directories for compiled/cache Smarty3 files

Maybe this fixes missing array element

So simple ... Ops!

Changes:
- enabled sub-directories for compiled/cached files which avoids large+slow
  directory descriptors
- changed some double-quotes to single

Merge pull request #11674 from nupplaphil/dep/guzzle

Bump guzzlehttp/guzzle from 6.5.5 to 6.5.8

Merge pull request #9 from nupplaphil/dependabot/composer/guzzlehttp/guzzle-6.5.8

Bump guzzlehttp/guzzle from 6.5.5 to 6.5.8

Merge pull request #11673 from Quix0r/bugs/returned-type-array-bool

Fixed: ?? didn't work here as bool won't be seen as null

Changed:
- empty() is maybe superflous here, still I would prefer a code style that is
  written explicitly and not rely on "magic casting"

Converted multiple single-comment (//) to multi-line comment block (/* */)

Images::getInfoFromURL[Cached]() will both return empty arrays on error and
that needs to be reflected here, too.

Fixed: ?? didn't work here as bool won't be seen as null

Merge pull request #11669 from MrPetovan/bug/11666-email-subject-double-encode

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

Fixes/more type hints 004

Fixed:
- TypeError: "Argument 1 passed to Friendica\Util\XML::escape() must be of the
  type string, null given, called in src/Util/XML.php line 171

Changes:
- added some documentation
- fixed some documentation
- changed more double-quotes to singl

Changes:
- added more type-hints
- added missing documentation
- Email::send() now returns what mail() returns (bool)

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

Return empty array on error (I hope it works this way).

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

Changes:
- addHubLink()'s 2nd parameter is DOMElement
- added more type-hints
- converted double-quotes to single

$data can turn into bool here

Ops, forgot to rename these, too.

Changes
- added type-hints
- added some documentation
- added default: block for unknown 'rel' value
- changed return type void to empty array

Changes:
- added type-hints
- added documentation

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

Merge pull request #11671 from MrPetovan/bug/fatal-errors

Add expected type-hint to BoundariesPager::renderFull

Add expected type-hint to BoundariesPager::renderFull

- Address https://github.com/friendica/friendica/issues/11630#issuecomment-1162634199

Remove type-hint inconsistent with expected return value in Database->getVariable

Remove email subject encoding from ItemCCEmail constructor

- It was wrongly re-encoded in Emailer::send

Merge pull request #11668 from Quix0r/renames/logger

Renamings in logger class to make them more clear

Changes:
- renamed Logger::getLogger() to getInstance() (@MrPetovan)

Bump guzzlehttp/guzzle from 6.5.5 to 6.5.8

Bumps [guzzlehttp/guzzle](https://github.com/guzzle/guzzle) from 6.5.5 to 6.5.8.
- [Release notes](https://github.com/guzzle/guzzle/releases)
- [Changelog](https://github.com/guzzle/guzzle/blob/6.5.8/CHANGELOG.md)
- [Commits](https://github.com/guzzle/guzzle/compare/6.5.5...6.5.8)

---
updated-dependencies:
- dependency-name: guzzlehttp/guzzle
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Changes:
- added type-hints
- added missing documentation
- renamed Logger::getWorker() to Logger::getLogger() as there is no worker class
  returned but the actual (inner) logger

Merge pull request #11665 from Quix0r/rewrites/dba-array-string-table

Rewrites/dba array string table

Added important comment from @nupplaphil@github.com

Reverted and improved TODO:
------
@nupplaphil: There is no easy possibility to add a logger here, that's why there
isn't any yet and instead a placeholder.. This execution point is a critical
state during a testrun, and tbh I'd like to leave here no further logic (yet)
because I spent hours debugging cases, where transactions weren't fully closed
and strange/unpredictable errors occur (sometimes -mainly during debugging other
errors :) ...)
-----

Fixed documentation

Rewrote to suggestions from @MrPetovan (I tried to maintain the old way of
['scheme' => 'table'] with my version).

Use protected $this->logger instead

Ops, autorun.sh is there but looks like lesser "multi-project" friendly as
MYSQL_* must be defined. I might still use my version as that is
project-specific.

Ops, WWORD ...

Added simple wrapper script to run PHPUnit tests for you. To use this, you need
to set at least 4 variables in your ~/.bashrc (recommended):

export FRIENDICA_MYSQL_HOST="localhost"
export FRIENDICA_MYSQL_DATABASE="friendica_test"
export FRIENDICA_MYSQL_USER="friendica_test"
export FRIENDICA_MYSQL_PASSWWORD="friendica_test"

Then you can put ./bin/dev/run_tests.sh into your .git/hooks/pre-commit file

Fixed:
- PHPUnit\Framework\Error\Notice:  Undefined index: language in /.../src/Util/EMailer/MailBuilder.php on line 122

Changes:
- let's at least log the error message, yes it does flood your logfile

Changes:
- Database->$driver can no longer be NULL, an empty string is fine anyway

Fixed:
- since $table can be now schema.table, this needs being taken care off
- maybe this foreach(explode()) is a bit an overkill as only 1 or 2 entries
  can be found in it

WIP: Properly some fixes? Also why is DROP VIEW IF EXISTS is being killed?