Results 1 to 2 of 2

Thread: Bug in email text

  1. #1
    Thread Starter
    henrikg's Avatar
    Join Date
    Dec 2011
    Location
    Denmark
    Posts
    97
    Thanks
    0
    Thanked 8 Times in 6 Posts

    Bug in email text

    Hi.

    There is a bug in the email text.

    The use of the text "Your %s Team", is supposed to use the blogname but this is implemented wrong in these files:

    includes/theme-cron.php
    includes/theme-emails.php
    includes/theme-options.php

    It is implemented like this:

    Code:
    $message .= sprintf( __( 'Your %s Team', APP_TD ), $blogname )
    This causes the language text "Your %s Team" to always be displayed like Your ClassiPress Team, and not using the blogname option, because the constant "ClassiPress" is insterted in the placeholder before the blogname gets a chance.

    It should have been implemented like this.

    Code:
    $message .= __( 'Your %s Team', $blogname ? $blogname : APP_TD )
    But honestly these messages should not be hardcoded in the php files, it should be options in the wp_admin.

  2. #2
    laura's Avatar
    Join Date
    Apr 2010
    Location
    Michigan
    Posts
    2,594
    Thanks
    60
    Thanked 279 Times in 254 Posts
    You must be an AppThemes customer and logged in to view this response. Join today!

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Similar Threads

  1. [SOLVED] Change email text?
    By cyclebarter in forum ClassiPress General Discussion
    Replies: 3
    Last Post: September 23rd, 2012, 10:30 AM
  2. Change email text?
    By cyclebarter in forum Report ClassiPress Bugs
    Replies: 1
    Last Post: September 22nd, 2012, 10:59 AM
  3. reminder email text
    By luukratief in forum Report JobRoller Bugs
    Replies: 1
    Last Post: June 22nd, 2011, 09:18 AM