WordPress.org

Plugin Directory

Changeset 175478


Ignore:
Timestamp:
11/20/09 07:35:58 (4 years ago)
Author:
brandondove
Message:

Fixed a few display and data related bugs.

Location:
conversation-starter
Files:
8 edited
3 copied

Legend:

Unmodified
Added
Removed
  • conversation-starter/tags/1.2/PluginCore/core.php

    r175055 r175478  
    2828                if (!is_admin()) : 
    2929                    wp_enqueue_style($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_css'); 
    30                     wp_enqueue_script($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_js', array('jquery')); 
     30                    wp_enqueue_script('jquery'); 
     31                    // wp_enqueue_script($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_js&convo-id='.$post->ID, array('jquery')); 
    3132                endif; 
    3233            } 
  • conversation-starter/tags/1.2/PluginCore/extend.php

    r175055 r175478  
    126126            function padding () { 
    127127                return array( 
    128                     'top' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    129                     'right' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    130                     'bottom' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    131                     'left' => esc_html_e(get_option($this->shortname.'_div_padding')), 
     128                    'top' => esc_html__(get_option($this->shortname.'_div_padding')), 
     129                    'right' => esc_html__(get_option($this->shortname.'_div_padding')), 
     130                    'bottom' => esc_html__(get_option($this->shortname.'_div_padding')), 
     131                    'left' => esc_html__(get_option($this->shortname.'_div_padding')), 
    132132                ); 
    133133            } 
    134134            function margin () { 
    135135                return array( 
    136                     'top' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    137                     'right' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    138                     'bottom' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    139                     'left' => esc_html_e(get_option($this->shortname.'_div_margin')), 
     136                    'top' => esc_html__(get_option($this->shortname.'_div_margin')), 
     137                    'right' => esc_html__(get_option($this->shortname.'_div_margin')), 
     138                    'bottom' => esc_html__(get_option($this->shortname.'_div_margin')), 
     139                    'left' => esc_html__(get_option($this->shortname.'_div_margin')), 
    140140                ); 
    141141            } 
     
    148148            } 
    149149            function borderColor () { 
    150                 return esc_html_e(get_option($this->shortname.'_border_color')); 
     150                return esc_html__(get_option($this->shortname.'_border_color')); 
    151151            } 
    152152            function borderType () { 
    153                 return esc_html_e(get_option($this->shortname.'_border_type')); 
     153                return esc_html__(get_option($this->shortname.'_border_type')); 
    154154            } 
    155155            function borderWidth () { 
    156                 return esc_html_e(get_option($this->shortname.'_border_width')); 
     156                return esc_html__(get_option($this->shortname.'_border_width')); 
    157157            } 
    158158            function backgroundColor () { 
    159                 return esc_html_e(get_option($this->shortname.'_background_color')); 
     159                return esc_html__(get_option($this->shortname.'_background_color')); 
    160160            } 
    161161            function fontColor() { 
    162                 return esc_html_e(get_option($this->shortname.'_font_color')); 
     162                return esc_html__(get_option($this->shortname.'_font_color')); 
    163163            } 
    164164            function fontWeight() { 
    165                 return esc_html_e(get_option($this->shortname.'_font_weight')); 
     165                return esc_html__(get_option($this->shortname.'_font_weight')); 
    166166            } 
    167167            function defaultPromptText() { 
    168                 return esc_html_e(get_option($this->shortname.'_default_prompt_text')); 
     168                return esc_html__(get_option($this->shortname.'_default_prompt_text')); 
    169169            } 
    170170        } 
  • conversation-starter/tags/1.2/javascripts/frontend.php

    r175122 r175478  
    11<?php 
    22    header("Content-type: text/javascript"); 
    3     global $post; 
    4     if (get_post_meta($post->ID, 'prompt', true) != '') : 
     3    if (get_post_meta(esc_attr__($_GET['convo-id']), 'prompt', true) != '') : 
    54?> 
    65jQuery(document).ready(function($) { 
     
    87        .map(function () { 
    98            if (this.tagName == "FORM") 
    10                 $(this).before('<div id="pj-convo-comment-prompt"><?php esc_attr_e(get_post_meta($post->ID, 'prompt', true)); ?></div>'); 
     9                $(this).before('<div id="pj-convo-comment-prompt"><?php esc_attr_e(get_post_meta(esc_attr__($_GET['convo-id']), 'prompt', true)); ?></div>'); 
    1110        }) 
    1211}); 
  • conversation-starter/tags/1.2/pj-convo.php

    r175123 r175478  
    55Description: This plugin prompts readers to answer a question in your comments. 
    66Author: brandondove, lordleiter, STDestiny, vegasgeek, drewstrojny 
    7 Version: 1.1.1 
     7Version: 1.2 
    88Author URI: http://www.think-press.com 
    99 
     
    8989    add_action('activate_conversation-starter/pj-convo.php', array(&$convo_starter, 'activateMe')); 
    9090    add_action('wp_head', 'pj_convo_starter_head_intercept'); 
     91    add_action('wp_footer', 'pj_convo_starter_foot_intercept'); 
    9192    function pj_convo_starter_head_intercept()  
    9293    { 
    9394        echo '<meta name="generator" content="Think-Press, Conversation Starter v1.1" />'; 
     95    } 
     96    function pj_convo_starter_foot_intercept () 
     97    { 
     98        global $post; 
     99        echo '<script language="javascript" src="'.get_option('home').'/index.php?conversation-starter=frontend_js&convo-id='.$post->ID.'"></script>';   
    94100    } 
    95101     
     
    102108        if (array_key_exists('conversation-starter', $wp->query_vars)  
    103109                && $wp->query_vars['conversation-starter'] == 'frontend_css') { 
    104  
    105110            include(CONVO_DIR.'/stylesheets/frontend.php'); 
    106111            die(); 
     
    115120    function pj_convo_query_vars($vars) { 
    116121        $vars[] = 'conversation-starter'; 
     122        $vars[] = 'convo-id'; 
    117123        return $vars; 
    118124    } 
  • conversation-starter/tags/1.2/readme.txt

    r175123 r175478  
    55Requires at least: 2.8 
    66Tested up to: 2.8.6 
    7 Stable tag: 1.1.1 
     7Stable tag: 1.2 
    88 
    99== Description == 
     
    4747 
    4848Stopped the box from showing up when no message was defined for a post (posts written before the plugin was installed) 
     49 
     50= Version 1.2 = 
     51 
     521. Fixed cases when incorrect prompt would show up above the comment form 
     532. Fixed several escaping functions that led to data being placed in incorrect places. 
  • conversation-starter/trunk/PluginCore/core.php

    r175055 r175478  
    2828                if (!is_admin()) : 
    2929                    wp_enqueue_style($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_css'); 
    30                     wp_enqueue_script($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_js', array('jquery')); 
     30                    wp_enqueue_script('jquery'); 
     31                    // wp_enqueue_script($this->shortname.'-public', get_option('home').'/index.php?'.$this->shortname.'=frontend_js&convo-id='.$post->ID, array('jquery')); 
    3132                endif; 
    3233            } 
  • conversation-starter/trunk/PluginCore/extend.php

    r175055 r175478  
    126126            function padding () { 
    127127                return array( 
    128                     'top' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    129                     'right' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    130                     'bottom' => esc_html_e(get_option($this->shortname.'_div_padding')), 
    131                     'left' => esc_html_e(get_option($this->shortname.'_div_padding')), 
     128                    'top' => esc_html__(get_option($this->shortname.'_div_padding')), 
     129                    'right' => esc_html__(get_option($this->shortname.'_div_padding')), 
     130                    'bottom' => esc_html__(get_option($this->shortname.'_div_padding')), 
     131                    'left' => esc_html__(get_option($this->shortname.'_div_padding')), 
    132132                ); 
    133133            } 
    134134            function margin () { 
    135135                return array( 
    136                     'top' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    137                     'right' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    138                     'bottom' => esc_html_e(get_option($this->shortname.'_div_margin')), 
    139                     'left' => esc_html_e(get_option($this->shortname.'_div_margin')), 
     136                    'top' => esc_html__(get_option($this->shortname.'_div_margin')), 
     137                    'right' => esc_html__(get_option($this->shortname.'_div_margin')), 
     138                    'bottom' => esc_html__(get_option($this->shortname.'_div_margin')), 
     139                    'left' => esc_html__(get_option($this->shortname.'_div_margin')), 
    140140                ); 
    141141            } 
     
    148148            } 
    149149            function borderColor () { 
    150                 return esc_html_e(get_option($this->shortname.'_border_color')); 
     150                return esc_html__(get_option($this->shortname.'_border_color')); 
    151151            } 
    152152            function borderType () { 
    153                 return esc_html_e(get_option($this->shortname.'_border_type')); 
     153                return esc_html__(get_option($this->shortname.'_border_type')); 
    154154            } 
    155155            function borderWidth () { 
    156                 return esc_html_e(get_option($this->shortname.'_border_width')); 
     156                return esc_html__(get_option($this->shortname.'_border_width')); 
    157157            } 
    158158            function backgroundColor () { 
    159                 return esc_html_e(get_option($this->shortname.'_background_color')); 
     159                return esc_html__(get_option($this->shortname.'_background_color')); 
    160160            } 
    161161            function fontColor() { 
    162                 return esc_html_e(get_option($this->shortname.'_font_color')); 
     162                return esc_html__(get_option($this->shortname.'_font_color')); 
    163163            } 
    164164            function fontWeight() { 
    165                 return esc_html_e(get_option($this->shortname.'_font_weight')); 
     165                return esc_html__(get_option($this->shortname.'_font_weight')); 
    166166            } 
    167167            function defaultPromptText() { 
    168                 return esc_html_e(get_option($this->shortname.'_default_prompt_text')); 
     168                return esc_html__(get_option($this->shortname.'_default_prompt_text')); 
    169169            } 
    170170        } 
  • conversation-starter/trunk/javascripts/frontend.php

    r175122 r175478  
    11<?php 
    22    header("Content-type: text/javascript"); 
    3     global $post; 
    4     if (get_post_meta($post->ID, 'prompt', true) != '') : 
     3    if (get_post_meta(esc_attr__($_GET['convo-id']), 'prompt', true) != '') : 
    54?> 
    65jQuery(document).ready(function($) { 
     
    87        .map(function () { 
    98            if (this.tagName == "FORM") 
    10                 $(this).before('<div id="pj-convo-comment-prompt"><?php esc_attr_e(get_post_meta($post->ID, 'prompt', true)); ?></div>'); 
     9                $(this).before('<div id="pj-convo-comment-prompt"><?php esc_attr_e(get_post_meta(esc_attr__($_GET['convo-id']), 'prompt', true)); ?></div>'); 
    1110        }) 
    1211}); 
  • conversation-starter/trunk/pj-convo.php

    r175123 r175478  
    55Description: This plugin prompts readers to answer a question in your comments. 
    66Author: brandondove, lordleiter, STDestiny, vegasgeek, drewstrojny 
    7 Version: 1.1.1 
     7Version: 1.2 
    88Author URI: http://www.think-press.com 
    99 
     
    8989    add_action('activate_conversation-starter/pj-convo.php', array(&$convo_starter, 'activateMe')); 
    9090    add_action('wp_head', 'pj_convo_starter_head_intercept'); 
     91    add_action('wp_footer', 'pj_convo_starter_foot_intercept'); 
    9192    function pj_convo_starter_head_intercept()  
    9293    { 
    9394        echo '<meta name="generator" content="Think-Press, Conversation Starter v1.1" />'; 
     95    } 
     96    function pj_convo_starter_foot_intercept () 
     97    { 
     98        global $post; 
     99        echo '<script language="javascript" src="'.get_option('home').'/index.php?conversation-starter=frontend_js&convo-id='.$post->ID.'"></script>';   
    94100    } 
    95101     
     
    102108        if (array_key_exists('conversation-starter', $wp->query_vars)  
    103109                && $wp->query_vars['conversation-starter'] == 'frontend_css') { 
    104  
    105110            include(CONVO_DIR.'/stylesheets/frontend.php'); 
    106111            die(); 
     
    115120    function pj_convo_query_vars($vars) { 
    116121        $vars[] = 'conversation-starter'; 
     122        $vars[] = 'convo-id'; 
    117123        return $vars; 
    118124    } 
  • conversation-starter/trunk/readme.txt

    r175123 r175478  
    55Requires at least: 2.8 
    66Tested up to: 2.8.6 
    7 Stable tag: 1.1.1 
     7Stable tag: 1.2 
    88 
    99== Description == 
     
    4747 
    4848Stopped the box from showing up when no message was defined for a post (posts written before the plugin was installed) 
     49 
     50= Version 1.2 = 
     51 
     521. Fixed cases when incorrect prompt would show up above the comment form 
     532. Fixed several escaping functions that led to data being placed in incorrect places. 
Note: See TracChangeset for help on using the changeset viewer.