Jquery replace not working

Question

Hi I am trying to change the contents of a hidden input.

var temp = $("#header").val().replace("(/<\/div><div>/g","<br>");

I tried using the following replace but it does not change it.

May I know what I did wrong? I am trying to replace:

</div><div>

to

<br>

EDIT - this is my program flow

User enters input to a wysiwyg text editor which produces the output
Save the user input to database
Go to another page where the wysiwyg input is retrieved from database and save it to input type hidden
Replace the </div><div> to a <br> which is then outputted to a div

I want to output it to a pdf using jspdf, but it does not understand HTML codes, and I am planning to output the line separately after a split. — RickyHalim
– RickyHalim, Commented Dec 11, 2014 at 12:34
what is an example of the contents you are trying to modify? and what is the expected result? You want to keep the first opening div and the last closing div? — wirey00
– wirey00, Commented Dec 11, 2014 at 12:34
So you are currently logging temp to the console or something and that's how you know the replace isn't working? Or...? — nnnnnn
– nnnnnn, Commented Dec 11, 2014 at 12:35

Javier del Saz · Accepted Answer · 2014-12-11 12:36:56Z

1

To replace all ocurrences in a string, the correct syntax with regExp is:

var temp = $("#header").val().replace(/<\/div><div>/g,"<br>");

You can test it, here:

'</div><div></div><div></div><div></div><div>'.replace(/<\/div><div>/g,"<br>");

returns:

<br><br><br><br>

edited Dec 11, 2014 at 12:36

answered Dec 11, 2014 at 12:31

Javier del Saz

8561 gold badge9 silver badges16 bronze badges

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

6 Comments

Drown Over a year ago

It should be noted that this only replaces the first occurence of </div><div>, you'll have to use Gaurang Tandon's solution to replace them all.

RickyHalim Over a year ago

Yes as Drown said, it only replaces the first occurence, and I need to replace all

RickyHalim Over a year ago

Thank you Javier! I suppose it didn't work because I put a " on the Regexp

Javier del Saz Over a year ago

you are welcome :), as you see if you quoted the regExp y treated as normal string

ravithejag Over a year ago

$(function(){ var text="132|233|424"; var temp = text.replace("|",","); alert(temp); }); expected result is 132,233,424 but its returning 132,233|424

|

putvande · Accepted Answer · 2014-12-11 12:31:14Z

0

You should do:

.replace(/<\/div><div>/g,"<br>");

The RegExp should not be enclosed within quotes else it would be a String. Also, the .val() method is used for textareas. Use .html() for header, divs, etc.

edited Dec 11, 2014 at 12:31

putvande

15.3k3 gold badges37 silver badges51 bronze badges

answered Dec 11, 2014 at 12:27

Gaurang Tandon

6,89911 gold badges52 silver badges96 bronze badges

1 Comment

RickyHalim Over a year ago

Hi, thanks for the response. Sorry I was using input type hidden not textarea, I wrote wrong. However after I change to your code, it still does not work

Collectives™ on Stack Overflow

Jquery replace not working

2 Answers 2

6 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related