Changeset 1035581

Timestamp:

11/30/2014 07:21:30 PM (11 years ago)

Author:

yingles

Message:

Fixes issue with not calculating the correct reading time when using more tags

Location:

reading-time-wp/trunk

Files:

• readme.txt (modified) (3 diffs)
• rt-reading-time.php (modified) (5 diffs)

reading-time-wp/trunk/readme.txt

@@ -5 +5 @@
 Requires at least: 3.0.1
 Tested up to: 4.0
-Stable tag: 1.0.0
+Stable tag: 1.0.2
 License: GPLv2 or later
 License URI: http://www.gnu.org/licenses/gpl-2.0.html
@@ -54 +54 @@
 == Changelog ==
 
+= 1.0.2 =
+* Fixing bug with more tags in the_content
+
 = 1.0.1 =
 * Converting the plugin to a class based structure
@@ -67 +70 @@
 = 1.0.1 =
 This update converts the plugin into a class based structure for better expandability in the future.
+
+= 1.0.2 =
+Fixes issue with miscalculated reading time when using <!--more--> tags

reading-time-wp/trunk/rt-reading-time.php

@@ -4 +4 @@
  * Plugin URI: http://jasonyingling.me/reading-time-wp/
  * Description: Add an estimated reading time to your posts.
- * Version: 1.0.1
+ * Version: 1.0.2
  * Author: Jason Yingling
  * Author URI: http://jasonyingling.me
@@ -29 +29 @@
     
     // Add label option using add_option if it does not already exist
+    public $readingTime;
     
     public function __construct() {
@@ -51 +52 @@
     }
     
+    public function rt_calculate_reading_time($rtPostID, $rtOptions) {
+        
+        $rtContent = get_post_field('post_content', $rtPostID);
+        $strippedContent = strip_shortcodes($rtContent);
+        $wordCount = str_word_count($strippedContent);
+        $this->readingTime = ceil($wordCount / $rtOptions['wpm']);
+        
+        return $this->readingTime;
+        
+    }
+    
     public function rt_reading_time($atts, $content = null) {
     
@@ -60 +72 @@
         $rtReadingOptions = get_option('rt_reading_time_options');
         
-        $rtPostID = get_the_ID();
-        $rtContent = get_the_content($rtPostID);
-        $strippedContent = strip_shortcodes($rtContent);
-        $wordCount = str_word_count($strippedContent);
-        $readingTime = ceil($wordCount / $rtReadingOptions['wpm']);
+        $rtPost = get_the_ID();
+        
+        $this->rt_calculate_reading_time($rtPost, $rtReadingOptions);
         
         return "
-        <span class='span-reading-time'>$label $readingTime $postfix</span>
+        <span class='span-reading-time'>$label $this->readingTime $postfix</span>
         ";
     }
@@ -82 +92 @@
     // Calculate reading time by running it through the_content
     public function rt_add_reading_time_before_content($content) {
+        $rtReadingOptions = get_option('rt_reading_time_options');
+
+        $originalContent = $content;
+        $rtPost = get_the_ID();
         
-        $rtReadingOptions = get_option('rt_reading_time_options');
-        
-        $originalContent = $content;
-        $strippedContent = strip_shortcodes($originalContent);
-        $wordCount = str_word_count($strippedContent);
-        $readingTime = ceil($wordCount / $rtReadingOptions['wpm']);
-        
+        $this->rt_calculate_reading_time($rtPost, $rtReadingOptions);
+                
         $label = $rtReadingOptions['label'];
         $postfix = $rtReadingOptions['postfix'];
         
-        $content = '<div class="rt-reading-time">'.'<span class="rt-label">'.$label.'</span>'.'<span class="rt-time">'.$readingTime.'</span>'.'<span class="rt-label"> '.$postfix.'</span>'.'</div>';
+        $content = '<div class="rt-reading-time">'.'<span class="rt-label">'.$label.'</span>'.'<span class="rt-time">'.$this->readingTime.'</span>'.'<span class="rt-label"> '.$postfix.'</span>'.'</div>';
         $content .= $originalContent;
         return $content;