Не отрабатывает replace?

Question

[SVM]

при клике по ссылке

<a href="http://coub.com/view/xwhu2" onclick="return frame(this)"></a>

её нужно вставить в атрибут src элемента с id="frame" , при этом заменить "view" на "embed"
что не правильно/почему не отрабатывает str.replace('/view/','/embed/');

function frame(arr){
        el=document.getElementById('frame');
        str=arr.href;
        str.replace('/view/','/embed/');
        str+='?muted=false&autostart=true&originalSize=false&startWithHD=false'+' '+"allowfullscreen=true" frameborder="0" width="360" height="180";
        el.src=str;
        return false;
}

p.s.
"методом научного тыка" разобрался
str=str.replace('/view/','/embed/');

Answer 1

[Сергей Мелодин]

1. Получить значение ссылки click
2. Заменить значение view на embed
3. Заменить src через attr()

Не нативно, зато под задачу.

UPD: https://codepen.io/anon/pen/KymMJw

Comments

[SVM]

"методом научного тыка" разобрался
str=str.replace('/view/','/embed/');

[Сергей Мелодин]

sotvm, можно и без косой черты

[SVM]

Сергей Мелодин,
так надёжнее
а вдруг попадёт такая ссылка coub .com/view/view
есть ещё вопрос ,подскажешь?
я кодер ещё тот , Билл Гейтс нервно курит в сторонке (͡๏̯͡๏).
не соображу как вставить значение html параметра,которое содержит двойные кавычки,
через регулярку в php вытаскиваю из JSON значение title = $2
пример:

$replacement = "<div date-title='$2' class='coub'><a href='//coub.com/view/\\${1}1'  onclick='return frame(this)'><img src='\$3small\$4' >$2</a></div>";

но когда оно содержит 'ка "вы" чки' то ломается вывод заголовка,
хрен бы с ним ,можно ведь вообще обойтись без этого атрибута,
но я не знаю ,как его можно "прочитать из ссылки",
т.е. нужно повесить onmouseover на этот div или img
и "читать/выводить" заголовок беря его из ссылки
мне он нужен только по событию т.к. сама ссылка скрыта (overflow: hidden;)
видно только саму картинку

[Сергей Мелодин]

sotvm, переменная читается в "лапках" и не читается в 'апострофах', поэтому или так:

$replacement = "<div date-title=$2 class=coub><a href=//coub.com/view/\\${1}1 onclick='return frame(this)'><img src=\$3small\$4 >$2</a></div>";

Или так:

$replacement = '<div date-title="'.$2.'" class=coub><a href=//coub.com/view/\\'.${1}1.' onclick="return frame(this)"><img src=\'.$3.'small\'.$4.' >'.$2.'</a></div>';

----

Чтобы было понятно: пихать в html кавычки можно, когда идёт текст с пробелами. В остальных случаях их можно опустить (HTML5 позволяет) и браузер сам разберётся.
В таком случае такой html-код: <a href=link class="class1 class2">phpVar</a> может быть выведен из php только с конкатенацией:

echo '<a href=link class="class1 class2">'.$someVar.'</a>'; // выведет: <a href="link" class="class1 class2">phpVar</a>

Когда твой html не имеет кавычек, то можно обойтись "лапками":

echo "<a href=link class=class1>$someVar</a>"; // выведет: <a href="link" class="class1">phpVar</a>

Answer 2

[Interface]

replace не мутирующий - он возвращает новую строку не меняя старую. Посмотрите на его работу в консоли

Comments

[SVM]

ну так как его мутировать ? ㋛ как сделать правильно?
как посмотретьв консоли ,увы тоже не знаю (͡๏̯͡๏)

[SVM]

ps
"методом научного тыка" разобрался
str=str.replace('/view/','/embed/');