Continued with coding convention:
- added curly braces around conditional code blocks
- added space between if/foreach/... and brace
- rewrote a code block so if dbm::is_result() fails it will abort, else the id
  is fetched from INSERT statement
- made some SQL keywords upper-cased and added back-ticks to columns/table names

Signed-off-by: Roland Haeder <roland@mxchange.org>

Continued with coding convention:
- added curly braces around conditional code blocks
- added space between if/foreach/... and brace
- rewrote a code block so if dbm::is_result() fails it will abort, else the id
  is fetched from INSERT statement
- made some SQL keywords upper-cased and added back-ticks to columns/table names

Signed-off-by: Roland Haeder <roland@mxchange.org>

Continued with coding convention:
- added curly braces around conditional code blocks
- added space between if/foreach/... and brace
- rewrote a code block so if dbm::is_result() fails it will abort, else the id
  is fetched from INSERT statement
- made some SQL keywords upper-cased and added back-ticks to columns/table names

Signed-off-by: Roland Haeder <roland@mxchange.org>

added more curly braces + space between "if" and brace

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
mod/admin.php

Don't cherry-pick:
- reverted dbm::is_result() to count()

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

Don't cherry-pick: reverted unrelated changes (dbm::is_result())

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

Only distribute items to active contacts

The object type "photo" is deprecated and was replaced by "image"

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
include/like.php
mod/photos.php
mod/subthread.php

Workaround for vanished database connections while authentication

Changed $a->get_baseurl() to App::get_baseurl()

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

Continued a bit:
- added more curly braces around conditional blocks
- added space between "if" and brace
- aligned "=>" (will do with more if wanted)

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

Opps, forgot this ...

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

added curly braces + space between "if" and brace

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

added much more curly braces + space between "if" and brace

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
mod/bookmarklet.php
mod/community.php
mod/contacts.php
mod/crepair.php
mod/events.php
mod/network.php
mod/suggest.php
mod/uexport.php
view/theme/duepuntozero/config.php
view/theme/frio/config.php
view/theme/quattro/config.php
view/theme/vier/config.php

Continued:

- added curly braces + spaces
- added todo about testing empty/non-empty strings

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

added spaces + curly braces

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

added spaces + curly braces

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

added more curly braces + fixed space->tab for code indentation.

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

added curly braces + space between "if" and brace

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
mod/notify.php

added curly braces + space between "if" and brace + initialized $result (was only within if() block but not outside of it)

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

added curly braces + space between "if" and brace + added missing @param

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

added curly braces + space between "if" and brace + added TODO

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

added curly braces + space between "if" and brace

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

Also here (was committed in other branch, too. :-( )

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

added more curyl braces + spaces between "if" and brace

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

Coding convention:
- added curly braces
- added space between "if" and brace
- also added TODO (old-lost code?)

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

Coding convention:
- added curly braces
- added space between "if" and brace

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

Coding convention applied:
- space between "if" and brace
- curly braces on conditional blocks

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
include/lock.php

*much* more usage of App::get_baseurl() instead of $a->get_baseurl() (coding convention applied)

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

No more needed  ... util/typo.php is there.

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

Use [] instead of test.

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

added lint.sh which needs to be executed like this:

$ ./util/lint.sh

No parameters needed. It will use "php -l" (current interpreter) for checking
for syntax errors in all PHP scripts.

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

Merge remote-tracking branch 'upstream/develop' into develop

Merge pull request #3010 from Quix0r/rewrites/mixed-static-object-reference-calls

Rewrites: mixed static object reference calls (and very sadly more ...)

Merge pull request #3026 from tobiasd/20161219-lng

DE translation of the core

DE translation of the core

Merge remote-tracking branch 'upstream/develop' into rewrites/mixed-static-object-reference-calls

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
include/follow.php

Merge pull request #3025 from tobiasd/20161219-messagespo

regenerated master messages.po file

regenerated master messages.po file

Merge pull request #3023 from annando/1612-attachment-link

Bugfix: Reimported Friendica posts from pump.io had the links missing.

Merge pull request #3024 from beardyunixer/develop

This error message had me confused for about twenty minutes.  I couldn't

This error message had me confused for about twenty minutes.  I couldn't
figure out what was wrong with my imagick install that would make it give
me an orange button if it was installed.  Of course, the problem was that
it wasn't installed, and the message was worded badly.  This is a bit less
confusing.

Bugfix: Reimported Friendica posts from pump.io had the links missing.

Merge pull request #3022 from annando/1612-ostatus-follow2

Ostatus: The follow/unfollow XML should now be even more compliant.

Mastodon: Hopefully we can now detect reshares better

Ostatus: The follow/unfollow XML should now be even more compliant.

Merge pull request #3021 from annando/1612-ostatus-follow

OStatus: Follow/Unfollow should now work again.

Removal of test code

OStatus: Follow/Unfollow should now work again.

Merge pull request #3014 from annando/1612-missing-self-contact

Create missing self contact

Convention: SQL keywords all upper-case, columns and table names (better) lower-cased and always in back-ticks.

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

Convention: SQL keywords all upper-case, columns and table names (better) lower-cased and always in back-ticks.

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

Garrr ...

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

Again, static *OR* object-referencing calls? Blue or red pill?

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

get_basepath() uses a lot $this (which means object-referenced calls, or
non-static calls). but still it is called statically in many places.

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

Again, static *OR* object-referencing calls? Blue or red pill?

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

get_basepath() uses a lot $this (which means object-referenced calls, or
non-static calls). but still it is called statically in many places.

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

Continued with #3010:
- more reverts of $a->get_baseurl()
- you always do: function foo (&$a), please read my TODO why it is not so good.
- for now I have fixed this inconsistency (compared to other method signatures)

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

Continued with #3010:
- more reverts of $a->get_baseurl()
- you always do: function foo (&$a), please read my TODO why it is not so good.
- for now I have fixed this inconsistency (compared to other method signatures)

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

Garrr ...

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

Opps, tpzo found ...

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

Opps, tpzo found ...

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

And more reverts:
- $a->get_baseurl() -> App::get_baseurl()
- was_recently_delayed() (entirely)

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

And more reverts:
- $a->get_baseurl() -> App::get_baseurl()
- was_recently_delayed() (entirely)

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

more reverts ...

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

more reverts ...

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

Also reverted these + added spaces for nicer appearance.

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

Also reverted these + added spaces for nicer appearance.

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

Create missing self contact

fixed SQL writing: keywords upper-case, column/table names in back-ticks. #3010

Signed-off-by: Roland Haeder <roland@mxchange.org>

fixed SQL writing: keywords upper-case, column/table names in back-ticks. #3010

Signed-off-by: Roland Haeder <roland@mxchange.org>

reverted change (which duplicated above query), thanks to @annano to point this out. #3010

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

reverted change (which duplicated above query), thanks to @annano to point this out. #3010

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

3 slashes for Doxygen, 2 are enough for PHP ... :-(

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

3 slashes for Doxygen, 2 are enough for PHP ... :-(

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

Continued with #3010:
- removed added debug messages (maybe they will come back in accordance to
  code-style guidelines?)
- converted $a->get_baseurl() back to App::get_baseurl()
- reverted back other code I have touched/merged as this was double:
  (substr($url, 0, 4) !== 'http') on $url = 'https://bla'; will be FALSE
  (means found http at start), too.

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

Continued with #3010:
- removed added debug messages (maybe they will come back in accordance to
  code-style guidelines?)
- converted $a->get_baseurl() back to App::get_baseurl()
- reverted back other code I have touched/merged as this was double:
  (substr($url, 0, 4) !== 'http') on $url = 'https://bla'; will be FALSE
  (means found http at start), too.

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

No more needed  ... util/typo.php is there.

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

Also removed this, seems to come in from wrong merging? #3010 Thanks to @annando

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

Opps, this has vanished by accident, thanks to @annando

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

Also removed this, seems to come in from wrong merging? #3010 Thanks to @annando

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

Opps, this has vanished by accident, thanks to @annando

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

Use [] instead of test.

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

added lint.sh which needs to be executed like this:

$ ./util/lint.sh

No parameters needed. It will use "php -l" (current interpreter) for checking
for syntax errors in all PHP scripts.

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

More dbm::is_result() instead of count()

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

Maybe no longer needed: converted $a->get_baseurl() to App::get_baseurl(). #3010

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

Opps, no && and AND in same line (confusing, thanks to @annando)

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

removed own debug lines (no more needed).

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

added extra spaces to make it look nice. #3010

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

Fixes for #3010:
- used static call, still App::get_baseurl() wraps it into an object-referencing
  call which seems to be a bit redundant?
- added spaces around dot (see one line below)

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

Cleanup for #3010 - added spaces, thanks to @annando .

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

Cleanup for #3010 - added spaces, thanks to @annando .

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

Cleanup for #3010 - added spaces, thanks to @annando .

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

Fixes for #3010 - removed baseurl as this is no longer done, still there are 89
left in templates.

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

Cleanup for #3010 - removed all redundant lines found by @rabuzarus

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

import dbm here to avoid adding everywhere the namespace (if later changed,
only here it needs change, cool ha? ;-)

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

Nope, get_basepath() should be static, then also mark it as static (in signature).

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

Merge branch 'rewrites/dbm_is_result' of github.com:Quix0r/friendica into rewrites/dbm_is_result

Signed-off-by: Roland Häder <roland@mxchange.org>
Conflicts:
include/Photo.php
include/api.php
include/cache.php
include/dbstructure.php
include/enotify.php
include/items.php
include/onepoll.php
include/queue_fn.php
include/session.php
include/text.php
mod/contacts.php
mod/dfrn_request.php
mod/display.php
mod/events.php
mod/item.php
mod/message.php
mod/msearch.php
mod/photos.php
mod/poco.php
mod/profile.php
mod/proxy.php
mod/videos.php
update.php
view/theme/diabook/theme.php
view/theme/vier/theme.php

More usage of dbm::is_result($r) instead of count($r):
- count() returns very different results and never a boolean (not even false on
  error condition).
- therefore you should NOT use it in boolean expressions. This still *can* be
  done in PHP because of its lazyness. But it is discouraged if it comes to
  more clean code.

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

Some rewrites:
- max_proccesses_reach() and maxload_reached() (why no _ behind max?) are called
  both way, static and with object reference.
- this is strongly discouraged and should be avoided as its support (in PHP) may
  be dropped in future releases.
- used $a = get_app(); to encapsulate code (even when the function does
  currently the same, it may be changed later)

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