No more needed (for this fix) but maybe later. So I always only comment them out. fixes/private_scope_on_tags
authorRoland Haeder <roland@mxchange.org>
Fri, 27 Mar 2015 23:10:06 +0000 (00:10 +0100)
committerRoland Haeder <roland@mxchange.org>
Fri, 27 Mar 2015 23:10:06 +0000 (00:10 +0100)
Signed-off-by: Roland Haeder <roland@mxchange.org>
classes/Notice.php
classes/Notice_tag.php
lib/tagcloudsection.php

index 9f0575228da0f745c37ec6b83b2feabb827463d5..263cc4e32394ecbcd2880ac2f8cc91e3067a9779 100644 (file)
@@ -2510,7 +2510,7 @@ class Notice extends Managed_DataObject
         $profile = Profile::current();
 
         // Is the general scope check okay and the user in logged in?
-        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . ']: inScope=' . intval($inScope) . ',profile[]=' . gettype($profile));
+        //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . ']: inScope=' . intval($inScope) . ',profile[]=' . gettype($profile));
         if (($inScope === TRUE) && ($profile instanceof Profile)) {
             /*
              * Check scope, else a privacy leaks happens this way:
@@ -2531,7 +2531,7 @@ class Notice extends Managed_DataObject
              * very political words.
              */
             $inScope = $this->inScope($profile);
-            /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . ']: inScope=' . intval($inScope) . ' - After inScope() has been called.');
+            //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . ']: inScope=' . intval($inScope) . ' - After inScope() has been called.');
         }
 
         $tags = array();
index 5e7631021eebd0910a1a8b2a1264fcd565b47cc7..d921b07594c4c99b9343ceb8bc376879466f64f9 100644 (file)
@@ -96,7 +96,7 @@ class Notice_tag extends Managed_DataObject
         $notice = new Notice();
         $notice->id = $this->notice_id;
 
-        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',this->notice_id=' . $this->notice_id . ' - Calling find() ... fetch() ...');
+        //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',this->notice_id=' . $this->notice_id . ' - Calling find() ... fetch() ...');
 
         // Fetch notice
         if ((!$notice->find()) || (!$notice->fetch())) {
@@ -104,7 +104,7 @@ class Notice_tag extends Managed_DataObject
             return FALSE;
         }
 
-        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',notice->id=' . $notice->id . ',notice->scope=' . $notice->scope);
+        //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',notice->id=' . $notice->id . ',notice->scope=' . $notice->scope);
 
         // Is it private scope?
         if ($notice->isPrivateScope()) {
@@ -114,17 +114,17 @@ class Notice_tag extends Managed_DataObject
             // Is the profile not set?
             if (!$profile instanceof Profile) {
                 // Public viewer shall not see a tag from a private dent (privacy leak)
-                /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] Not logged in, skipping ...');
+                //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] Not logged in (public view).');
                 $inScope = FALSE;
             } elseif (!$notice->inScope($profile)) {
                 // Current profile is not in scope (not allowed to see) of notice
-                /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] profile->id=' . $profile->id . ' is not allowed to see this tag, skipping ...');
+                //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] profile->id=' . $profile->id . ' is not allowed to see this tag.');
                 $inScope = FALSE;
             }
         }
 
         // Return result
-        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',this->weight=' . $this->weight . ',inScope=' . intval($inScope) . ' - EXIT!');
+        //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] this->tag=' . $this->tag . ',this->weight=' . $this->weight . ',inScope=' . intval($inScope) . ' - EXIT!');
         return $inScope;
     }
 }
index f82a0dd153189955507406de90cdd2b4f5b709b2..268540cf9be4a97193035ea82d9919acd06ce6b7 100644 (file)
@@ -50,7 +50,7 @@ class TagCloudSection extends Section
     function showContent()
     {
         $tags = $this->getTags();
-        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] tags[]=' . gettype($tags));
+        //* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . '] tags[]=' . gettype($tags));
 
         if (!$tags) {
             // TRANS: Content displayed in a tag cloud section if there are no tags.