Changeset 917703

Timestamp:

05/20/2014 06:38:05 AM (12 years ago)

Author:

kjbenk

Message:

Fixed Bug with Multiple Portfolios

File:

• show-stock-quotes/trunk/show_stock_quotes.php (modified) (5 diffs)

show-stock-quotes/trunk/show_stock_quotes.php

@@ -4 +4 @@
 Plugin URI: http://kylebenkapps.com/wordpress-plugins/
 Description: Show stock quotes updated in real-time.
-Version: 1.4.1
+Version: 1.5
 Author: Kyle Benk
 Author URI: http://kylebenkapps.com
@@ -49 +49 @@
     /** @see WP_Widget::widget */
     function widget($args, $instance) {
+        
         extract( $args );
+        
         $title = $instance['title'];
         
@@ -60 +62 @@
         }
         
-        $tickers = $this->kjb_get_stock_data($instance);
+        $tickers = array();
+        
+        for ($i = 1; $i < 21; $i++) {
+            
+            $ticker = $instance['stock_' . $i];
+            
+            if ($ticker != '') {
+                $tickers[] = $ticker;
+            }
+        }
+        
+        $this->kjb_get_stock_data($instance);
         
         //Display all stock data
@@ -143 +156 @@
                 $stock = isset($instance['stock_'.$i]) ? $instance['stock_'.$i] : '';
                 ?>
-                <li><input class="widefat" id="<?php echo $this->get_field_id( 'stock_'.$i ); ?>" name="<?php echo $this->get_field_name( 'stock_'.$i); ?>" type="text" value="<?php echo esc_attr( $stock ); ?>" /></li>
+                <li><input class="widefat" id="<?php echo $this->get_field_id( 'stock_'.$i ); ?>" name="<?php echo $this->get_field_name('stock_' . $i); ?>" type="text" value="<?php echo esc_attr( $stock ); ?>" /></li>
                 <?php
             }
@@ -161 +174 @@
         wp_enqueue_style('kjb_quotes_css_src');
         
-        wp_enqueue_script('kjb_jquery', "//code.jquery.com/jquery-1.10.2.js");
-        
-        //$out = get_transient('kjb_stockdata_transient');
-        
-        //if (false === $out){
-        for ($i = 1; $i < 21; $i++) {
-            $ticker = $ticker_info['stock_'.$i];
-            if ($ticker != '') {
-                //Get stock data
-                //$contents = str_getcsv(file_get_contents('http://download.finance.yahoo.com/d/quotes.csv?s='.$ticker.'&f=sl1c1c0&e=.csv'),',');
-                /*
-if ($contents[1] != '0.00') {
-                    $temp = array(
-                    'ticker' => $contents[],
-                    'quote' => $contents[],
-                    'change' => $contents[],
-                    'change_precent' => $contents[]
-                );
-                }else{
-                    $temp = 'Invalid ticker';
+        $out = array();
+        
+        $all_tickers = get_option('widget_kjb_show_stocks');
+        
+        if ($all_tickers !== false) {
+            
+            foreach ($all_tickers as $key => $val) {
+                
+                for ($i = 1; $i < 21; $i++) {
+            
+                    $ticker = $val['stock_' . $i];
+                    
+                    if ($ticker != '') {
+                        $out[] = $ticker;
+                    }
                 }
-*/                  
-                $out[] = $ticker;
             }
-        }
-        
-        $passed_data = array(
-            'out'                   => isset($out) ? $out : array(),
-            'quote_display_color'   => $ticker_info['quote_display_color']
-        );
+            
+            $passed_data = array(
+                'out'                   => isset($out) ? $out : array(),
+                'quote_display_color'   => $val['quote_display_color']
+            );
+        }
         
         wp_localize_script('kjb_quotes_js_src', 'passed_data', $passed_data);
-            
-            
-            //set_transient('kjb_stockdata_transient',$out,60); 
-        //}
         
         return isset($out) ? $out : array();