1

I'm working on an inline visualforce page that is embedded into a standard lead detail record. I'm using javascript logic on the vf page to get the value of a field and then, depending on the value retrieved, display different messages. The issue I'm a having is that I'm not able to test in the javascript whether the field is null/blank/''/"".

My code looks like: 

<div id="divFDNC"></div>

    <script language="javascript">

       if(document.getElementById('divFDNC')){
          var gIE = document.all ? true:false;
          if({!Lead.Checkbox__c} == true){

            if({!Lead.Custom_Status__c} == '00' || {!Lead.Custom_Status__c} == ''){
               document.getElementById('divFDNC').innerHTML = '<font style="font-size: 11px;" class="FDNC_Message">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<b>MY MESSAGE</b></font>'
             }
            else{
               document.getElementById('divFDNC').innerHTML = '<font style="font-size: 11px;" class="FDNC_Message">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<b>MY MESSAGE 2</b></font>'
             }

The message works when I enter '00' into the Custom_Status__c field. However, when the Custom_Status__c field is blank, then it doesn't display the message like it should. I've tried the following to get it to work (no luck on any of them):

'{!Lead.Custom_Status__c}' == '' / null / NaN / undefined

"{!Lead.Custom_Status__c}" == "" / null / NaN / undefined

{!Lead.Custom_Status__c} == '' / null / NaN / undefined

Does anyone know how to properly test that a field is blank, using javascript? Thanks in advance for any help you can provide.

Improve this question
4
  • In JS no need to use ==. Reserve that for Apex. Just test against "" for an empty string. Commented Sep 2, 2015 at 17:57
  • 1
    The check with quotes around '{!Load.Custom_Status__c}' should have worked as it works for me. Alternatively you can change it to if{!(ISBLANK(Lead.Custom_Status__c)} without any quotes. Commented Sep 2, 2015 at 18:10
  • I think it would need to be if ({!ISBLANK(...)}) with parentheses. Commented Sep 2, 2015 at 18:12
  • Thank you all for the responses. Still not working...not sure what's going on. Commented Sep 2, 2015 at 20:09

2 Answers 2

Reset to default
1

Anything that doesn't strictly evaluate to a native value should be enclosed in quotes, and you should also escape those values to avoid XSS injection attacks.

I would personally write the following code:

if({!OR(Lead.Custom_Status__c = '00', ISBLANK(Lead.Custom_Status__c))}) {

Salesforce will evaluate this to

if(true)

when the status is blank or set to 00, and

if(false)

otherwise.

Alternatively, if you want to use the value directly, make sure you use JSENCODE:

var customStatus = "{!JSENCODE(Lead.Custom_Status__c)}";
if(customStatus == "00" || !customStatus) {
Improve this answer
3
  • I would agree with you (and I do like your alternative suggestion of using the var), but it goes beyond what you're mentioning as a solution. I'm not a js devloper, but I found that one of the reasons my js wasn't working was because I was giving the field different values in my code. When trying to get null, I originally had '{field}' = '' || {field} = '00'. I found that it only worked when all of the {field}'s had the "" around them --> "{field}" = "" || "{field}" = '00'. Only then did all values get checked Commented Sep 3, 2015 at 19:02
  • @rwegner7 I recommend this technique doubly-so if JavaScript isn't your strong point. By having the system evaluate stuff to a native true/false value, you don't have to worry about how/if/where to quote, how quotes should work, etc. You can have salesforce.com evaluate the expression and give JS a simple Boolean to execute code from. I will admit, JS can be daunting, and so using formula expressions might be a more natural form to work with. Commented Sep 3, 2015 at 19:13
  • I'm definitely a newbie at js, I think the var method you mentioned is the strongest way to do it, ensures the most consistency. Thanks for your suggestions! :-) Commented Sep 3, 2015 at 19:35
-1

Simple answer is that Salesforce is just weird when it comes to javascript. Even though the Lead.Custom_Status__c field is a text field, for some reason the javascript had different requirements when reading number values in the field vs alphabetical values.

Here's what worked:

if("{!Lead.Custom_Status__c}" == '00' || "{!Lead.Custom_Status__c}" == "" || "{!Lead.Custom_Status__c}" == "AA")

Notice how for the number values you only need single quotes around the value (''), but for text and null you need double quotes (""). I'm not sure why, but that's what solved it.

Improve this answer
1
  • 1
    It may have worked, but you have no idea why it worked. This answer is pretty far off the mark. Both quotes have the same behavior in JavaScript. There's better ways to accomplish your goal. Commented Sep 3, 2015 at 18:09

You must log in to answer this question.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.