What the {if} am I doing wrong?

[BigOrange]

I'm trying to modify the code in step 3 of checkout to always list the items with ($0.00 ) cost that are included in a package as the default choice in the drop down list. But, I want the choice to default to "No Thanks" if there is an addition charge for the item. That way the free features that are included in the package are always selected for the user during checkout and not the ones that require extra payment.

The code below is for the "featured ad" list box but the logic should apply to the "bold" and "Highlighted" drop down lists as well.

Could someone look at the code below and see why it won't work? It always drops to the 'else' options. I have only changed the code between the comments.

Thanks!

	{if $featured=="Y"}
	<tr>
          <td class="formleft">{$smarty.const.LANG_ADD_FEATURED}{$smarty.const.LANG_COLON}</td>
          <td class="formright">
			     ({$cFeaturedPrice}) <select name="featured">
           <!-- Default to free features -->
           {if $cFeaturedPrice=="$0.00 "}
    	      <option value="Y">{$smarty.const.LANG_YES} (+{$cFeaturedPrice})</option>
              <option value="N">{$smarty.const.LANG_NO_THANKS}</option>
          {else}
              <option value="N">{$smarty.const.LANG_NO_THANKS}</option>
              <option value="Y">{$smarty.const.LANG_YES} (+{$cFeaturedPrice})</option>
          {/if}
          <!-- END Default to free features -->
			</select>
		</td>
	</tr>
	{/if}

[sally_b]

Hi. Maybe it is something to do with the $ and the extra blank space in the IF statement. Perhaps try

{if $cFeaturedPrice=="0.00"}

[BigOrange]

Originally Posted by sally_b

Hi. Maybe it is something to do with the $ and the extra blank space in the IF statement. Perhaps try

{if $cFeaturedPrice=="0.00"}

If I output $cFeaturedPrice to the screen the output is $0.00 with what appears to be a space beside it when highlighted. But with the if statement set up that way it does not work.

Thanks for your help! Any other suggestions are welcomed.

[Eric Barnes]

I would recommend putting {debug} var in that template then a smarty popup should appear with the value it is looking for.

[BigOrange]

Originally Posted by suzkaw

I would recommend putting {debug} var in that template then a smarty popup should appear with the value it is looking for.

Thanks Eric for motivating me to get back on this. I had pretty much given up.

Here was the problem: {if $cFeaturedPrice=="$0.00"}
Here was the solution: {if $cFeaturedPrice=='$0.00'}

Using single quotes instead of double made it work correctly. I'm surprised I wasn't getting an error message.

Thanks for your help!

[Eric Barnes]

Ahh yes. I didn't even look at the quotes.

With double quotes: "$0.00" it thinks the $ is a variable.
With single quotes: '$0.00' it doesn't.

You could probably also do this: "\$0.00" to escape the dollar sign.

[BigOrange]

I have a similar problem with the function.new_listings.php plugin. I'm trying to show a camera icon if a listing has photos and nothing if the listing has no photos. The variable I'm trying to use is $entry.image but I'm not sure it is correct. The code I have added has comments that end in RLB.

If you can offer any help it would be appreciated.


Thanks for the great support!

PHP Code:

    $loop_count = count($loop)-1;

    $output = "<table " . $table_attr.">\n";
    for ($x = 0; $x<= $loop_count; $x++)
    {
        //opening tr
        $output .= "<tr" . smarty_function_html_table_cycle_newlistings('tr',   $tr_attr, $c) . ">\n";
        //opening td
        $output .= "<td class=\"newlist\"" . smarty_function_html_table_cycle_newlistings('td',   $td_attr, $c) . ">\n";
            //title
            if(defined('SEO'))
            {
                $listing_title=$loop[$x][title];
                $listing_title=str_replace(' ', '_', $listing_title);
                $listing_title=preg_replace('/\W/e', '', $listing_title);
                $output.="<a href='listing/".$loop[$x][id]."/".$listing_title.".html'>".$loop[$x][title]."</a>";
            }

            // This is being used RLB
            else
            {
             // Check for photo in ad by RLB
               if ($entry.image != " ")
               {
                  $output.="<img src='/templates/default/images/ico_has_photo.jpg' align='left' width='18'> ";
               }
              else
              {
                  $output.=" ";
              }
            // END Check for photo in ad by RLB

              //ad title
              $output.="<a href='viewlisting.php?view=". $loop[$x][id] ."'>".$loop[$x][title]."</a>  ";
            }

              //price
              $output.="<center>"."<span style='font-size: 10px; color: #000000;'>".$loop[$x][price]."   (";
            
              //date
              //$output.=smarty_modifier_date_format($loop[$x][dateadded], $date_format);

              // city
              $output.=$loop[$x][city].")</span></center>";

              // city
              // $output.=$loop[$x][state].")";

              //closing td
              $output.="</td>\n";
              $output .= "</tr>\n";
    }
    $output .= "</table><br />\n"; 


[Eric Barnes]

I think this is your problem:
if ($entry.image != " ")

Should be:
if ($entry.image != "")

[BigOrange]

That doesn't work either. I'm wondering if $entry.image is what I should be testing for.

When I try doing the opposite test [ if ($entry.image == "") ] the camera Icon never shows, even for ads that have photos. It's like I'm testing for the wrong variable or one that is not available in the function.new_listings.php plugin. Any other ideas why it doesn't work?

PHP Code:

// Check for photo in ad by RLB
               if ($entry.image == "")
               {
                  $output.="<img src='/templates/default/images/ico_has_photo.jpg' align='left' width='18'> ";
               }
              else
              {
                  $output.=" ";
              }
            // END Check for photo in ad by RLB 


[Eric Barnes]

It will probably be easier if you just paste the whole file. I think what I need to see is not included with your first code paste.