Avoid jquery function for all the content

Question

I have a table with 20 rows. In each row there is an element

<p class="exp-date">'.$cust->Expiration_Date.'</p>

This element is going to be repeated and return different values but in a lot of rows return 0001-01-01. I want to hide this content so I wrote this in javascript

var exp = $(".exp-date").val();  
var exphide = '0001-01-01';
    if(exp = exphide) {
        $(".exp-date").html('');
    }

and also have tried this

$('.exp-date').each(function() {
        if(exp = exphide) {
            $(".exp-date").html('');
        }
    });

But in both cases apply the jquery on the first row and modify everything not only where the statement is declared.

Someone any idea?

Thanks in advance

Use Double equal to or Triple equal to. Refer this

Bugfixer
– Bugfixer

2015-08-31 04:45:45 +00:00
Commented Aug 31, 2015 at 4:45 — Bugfixer
– Bugfixer, Commented Aug 31, 2015 at 4:45

Tushar · Accepted Answer · 2015-09-02 04:18:49Z

3

You're using assignment in if statement. The condition exp = exphide will always evaluate to true and the code inside the if statement will execute for all the elements.

Change

if(exp = exphide) {

to

if(exp == exphide) {

or

if(exp === exphide) {

Also, use text() instead of html() to get the date, and use trim() on it to remove extra spaces before comparing.

Use this/$(this) inside the each to get the innerText of the current element.

$('.exp-date').each(function () {
    if ($(this).text().trim() == exphide) {
    //  ^^^^^^^^^^^^^^^^^^^^^^^^
        $(this).html('');
        // ^^^^
    }
});

edited Sep 2, 2015 at 4:18

answered Aug 31, 2015 at 4:39

Tushar

87.4k21 gold badges164 silver badges182 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Ram Over a year ago

Note that you are missing the this keyword! $(this).html('');, $(this).text(''), or better $(this).empty()

Tushar Over a year ago

@Vohuman I've added that in the last code snippet, can you please check again

Tomas Lucena Over a year ago

sorry man, dont work (continue applying for all p) but a guy gave me the solution!!! I will post it

Unni Babu · Accepted Answer · 2015-08-31 04:42:55Z

2

Use == and "this", else it will point to all classes. Code shown below

    var exphide = '0001-01-01';
    $('.exp-date').each(function() {
        if(this.innerHTML == exphide) {  //this.innerHTML
            $(this).html('');  //this.html..else will point to all classes
        }
    });

answered Aug 31, 2015 at 4:42

Unni Babu

1,81412 silver badges16 bronze badges

Comments

Murali Mohan · Accepted Answer · 2015-08-31 05:03:02Z

0

First you should correct the syntax in the if condition and then try the below code. When you are using "each" for looping you should pass index and value to the callback function as shown below. Then you can achieve the desired result.

var exphide = '0001-01-01';
$('.exp-date').each(function(index, value) {
        var exp = $(value).text(); 
        if(exp == exphide) {
            $(value).html('');
        }
    });

I suggest not to remove the content from table cell instead you can hide. Hope this helps.

answered Aug 31, 2015 at 5:03

Murali Mohan

1367 bronze badges

Collectives™ on Stack Overflow

Avoid jquery function for all the content

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related