Adhere feedback

[friendica-addons.git] / planets / planets.php

diff --git a/planets/planets.php b/planets/planets.php
index 6b7c2527e75c986ee1e609898e80ea7b687d5c50..8a206fb51641c2bd46df0f514b39a403b4fe9aaa 100644 (file)
--- a/planets/planets.php
+++ b/planets/planets.php
@@ -13,59 +13,56 @@ use Friendica\Core\Logger;
 use Friendica\Core\Renderer;
 use Friendica\DI;
 
-function planets_install() {
-
+function planets_install()
+{
        /**
-        *
         * Our demo addon will attach in three places.
         * The first is just prior to storing a local post.
-        *
         */
-
        Hook::register('post_local', 'addon/planets/planets.php', 'planets_post_hook');
 
        /**
-        *
         * Then we'll attach into the addon settings page, and also the
         * settings post hook so that we can create and update
         * user preferences.
-        *
         */
-
        Hook::register('addon_settings', 'addon/planets/planets.php', 'planets_settings');
        Hook::register('addon_settings_post', 'addon/planets/planets.php', 'planets_settings_post');
 
        Logger::notice("installed planets");
 }
 
-function planets_post_hook(App $a, &$item) {
-
-       /**
-        *
-        * An item was posted on the local system.
-        * We are going to look for specific items:
-        *      - A status post by a profile owner
-        *      - The profile owner must have allowed our addon
-        *
-        */
-
+/**
+ * An item was posted on the local system.
+ * We are going to look for specific items:
+ *      - A status post by a profile owner
+ *      - The profile owner must have allowed our addon
+ */
+function planets_post_hook(&$item)
+{
        Logger::notice('planets invoked');
 
-       if(! local_user())   /* non-zero if this is a logged in user of this system */
+       if (!DI::userSession()->getLocalUserId()) {
+               /* non-zero if this is a logged in user of this system */
                return;
+       }
 
-       if(local_user() != $item['uid'])    /* Does this person own the post? */
+       if (DI::userSession()->getLocalUserId() != $item['uid']) {
+               /* Does this person own the post? */
                return;
+       }
 
-       if($item['parent'])   /* If the item has a parent, this is a comment or something else, not a status post. */
+       if ($item['parent']) {
+               /* If the item has a parent, this is a comment or something else, not a status post. */
                return;
+       }
 
        /* Retrieve our personal config setting */
+       $active = DI::pConfig()->get(DI::userSession()->getLocalUserId(), 'planets', 'enable');
 
-       $active = DI::pConfig()->get(local_user(), 'planets', 'enable');
-
-       if(! $active)
+       if (!$active) {
                return;
+       }
 
        /**
         *
@@ -96,12 +93,13 @@ function planets_post_hook(App $a, &$item) {
  *
  */
 
-function planets_settings_post(App $a,$post) {
-       if (! local_user()) {
+function planets_settings_post($post)
+{
+       if (!DI::userSession()->getLocalUserId()) {
                return;
        }
        if ($_POST['planets-submit']) {
-               DI::pConfig()->set(local_user(), 'planets', 'enable' ,intval($_POST['planets']));
+               DI::pConfig()->set(DI::userSession()->getLocalUserId(), 'planets', 'enable' ,intval($_POST['planets']));
        }
 }
 
@@ -115,13 +113,13 @@ function planets_settings_post(App $a,$post) {
 
 
 
-function planets_settings(App &$a, array &$data)
+function planets_settings(array &$data)
 {
-       if(! local_user()) {
+       if(!DI::userSession()->getLocalUserId()) {
                return;
        }
 
-       $enabled = DI::pConfig()->get(local_user(),'planets','enable');
+       $enabled = DI::pConfig()->get(DI::userSession()->getLocalUserId(),'planets','enable');
 
        $t    = Renderer::getMarkupTemplate('settings.tpl', 'addon/planets/');
        $html = Renderer::replaceMacros($t, [