Change the innerHTML content using JQuery

Question

var user;
(function (user) {
    user = {
        dynamicPriceChange: function () {
            $("input[name='user[plan_id]']").change(function (e) {
                var planId = $(this).data('text');
                var durationPlan = $('p#durSubsMY')[0].innerHTML;
                var price = $('.price')[0].innerHTML;
                $('.price').text(planId);
                $('span#Love')[0].innerHTML = price;
                if price == "29" {
                    durationPlan = "per month";
                }
                if price == "261" {
                    durationPlan = "per year";
                }
            });
        }
        jQuery(function () {
            user.dynamicPriceChange();
        });
    })(user)

I'm trying to change durationPlan as "per month" if the price is 29 & "per year" if the price is 261.
But I'm unable to change that. Please help, I'm new to jQuery.

The Working Corrected Code is

    if (price == "29") {
         $('p#durSubsMY')[0].innerHTML = "per month"
        }
    if (price == "261" ){
         $('p#durSubsMY')[0].innerHTML = "per year"
        }

Thanks everybody for your help !!!

Cheers ! :-)

Also I would suggest using === instead of == stackoverflow.com/questions/359494/… — Marcel
– Marcel, Commented Dec 20, 2012 at 8:08

Musa · Accepted Answer · 2012-12-20 08:02:31Z

2

durationPlan is just a variable that contains the innerHTML of the element changing its value does not alter the elements content. Try instead

    if (price == "29") {
        $('p#durSubsMY')[0].innerHTML = "per month"
    }
    else if (price == "261") {
        $('p#durSubsMY')[0].innerHTML = "per year"
    }

edited Dec 20, 2012 at 8:02

answered Dec 20, 2012 at 7:43

Musa

97.9k17 gold badges123 silver badges144 bronze badges

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

Comments

Andrei Stalbe · Accepted Answer · 2012-12-20 07:49:04Z

2

As #Adil mentioned, you forgot you parenthesis in the if statement. Also you can try html() function from the jQuery library http://api.jquery.com/html/ instead of innerHTML.

var planId = $(this).data('text'); if you want to get the value from the input use var planId = $(this).val()

answered Dec 20, 2012 at 7:49

Andrei Stalbe

1,5417 gold badges27 silver badges46 bronze badges

Comments

Adil · Accepted Answer · 2012-12-20 07:50:25Z

2

Wrong syntax of if statement, you missed the if condition part parenthesis.

Change

if price == "29" {
       durationPlan = "per month"
}

if price == "261" { durationPlan = "per year" }

To

if (price == "29") {
      durationPlan = "per month"
}
if (price == "261" ){
     durationPlan = "per year"
}

One of closing bracket is also missing in the end.

Change

})(user)

To

}})(user)

edited Dec 20, 2012 at 7:50

answered Dec 20, 2012 at 7:42

Adil

148k25 gold badges217 silver badges207 bronze badges

4 Comments

povilasp Over a year ago

ALWAYS make a fiddle before posting a question in javascript :)

Cerbrus Over a year ago

OP also appears to be missing a closing bracket } at the end of his code. Might be nice to add to the answer.

RoR Prog Over a year ago

Adil its effecting my other functionality, i mean that i have written few more functions whose functionality got affected by writing this, how do i keep them unchanged. Sorry but how to get everything working ?

RoR Prog Over a year ago

ok....thanx adil & everybody here for such a quick response to solve my prob, i was stuck since yesterday. Thanks everyone....

Collectives™ on Stack Overflow

Change the innerHTML content using JQuery

3 Answers 3

Comments

Comments

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related