Changeset 486

Timestamp:

02/28/08 20:00:25 (10 months ago)

Author:

padams

Message:

added admin setting for disabling feed link tracking.

Location:

trunk

Files:

• modules/base/classes/settings.php (modified) (1 diff)
• modules/base/templates/options_general.tpl (modified) (1 diff)
• owa_wp.php (modified) (2 diffs)

trunk/modules/base/classes/settings.php

@@ -410 +410 @@
                         'cache_objects'                                 => true,
                         'do_not_log_admins'                             => true,
-                        'do_not_log_ips'                                => ''
+                        'do_not_log_ips'                                => '',
+                        'track_feed_links'                                      => true
                         
                         ));

trunk/modules/base/templates/options_general.tpl

@@ -135 +135 @@
         <BR>
 
+        <fieldset name="owa-feed-options" class="options">
+                <legend>Feed Tracking</legend>
+                
+                <div class="setting" id="feeds">        
+                        <div class="title">Feed Link Tracking</div> 
+                        <div class="description">Adds tracking parameters to RSS or Atom feeds links. This provides a way to track how many visitors come from your feeds.</div>
+                        <div class="field">
+                                <select name="<?=$this->getNs();?>config[track_feed_links]">
+        
+                                        <option value="0" <? if ($config['track_feed_links'] == false):?>SELECTED<?endif;?>>Off</OPTION>
+                                        <option value="1" <? if ($config['track_feed_links'] == true):?>SELECTED<?endif;?>>On</OPTION>
+                                </select>
+                        </div>
+                </div>
+                
+        </fieldset>
+        
+    <BR>
+        
     <fieldset name="owa-event-options" class="options">
                 <legend>Event Handling</legend>

trunk/owa_wp.php

@@ -59 +59 @@
                 // check for presence of '?' which is not present under URL rewrite conditions
         
-                if (strpos($link, "?") === false):
-                        // add the '?' if not found
-                        $link .= '?';
+                if ($this->config['track_feed_links'] == true):
+                
+                        if (strpos($link, "?") === false):
+                                // add the '?' if not found
+                                $link .= '?';
+                        endif;
+                        
+                        // setup link template
+                        $link_template = "%s&%s=%s&%s=%s";
+                                
+                        return sprintf($link_template,
+                                                   $link,
+                                                   $this->config['ns'].$this->config['source_param'],
+                                                   'feed',
+                                                   $this->config['ns'].$this->config['feed_subscription_param'],
+                                                   $_GET[$this->config['ns'].$this->config['feed_subscription_param']]);
+                else:
+                        return;
                 endif;
-                
-                // setup link template
-                $link_template = "%s&%s=%s&%s=%s";
-                        
-                return sprintf($link_template,
-                                           $link,
-                                           $this->config['ns'].$this->config['source_param'],
-                                           'feed',
-                                           $this->config['ns'].$this->config['feed_subscription_param'],
-                                           $_GET[$this->config['ns'].$this->config['feed_subscription_param']]);
-                                          
         }
         
@@ -84 +88 @@
         function add_feed_tracking($binfo) {
                 
-                $guid = crc32(getmypid().microtime());
+                if ($this->config['track_feed_links'] == true):
+                        $guid = crc32(getmypid().microtime());
                 
-                return $binfo."&".$this->config['ns'].$this->config['feed_subscription_param']."=".$guid;
+                        return $binfo."&".$this->config['ns'].$this->config['feed_subscription_param']."=".$guid;
+                else:
+                        return;
+                endif;
         }