setAttribute 被忽略，为什么？

Question

此代码的目的是将 HTML 属性“data-original-src”添加到 img-tag 中，以防不安全的 http:// URL。但是没有生成这个属性。

出了什么问题，我忽略了什么错误？ 无论如何，str_replace() 就像一个魅力。

public function parse($string) {
    if($this->settings['camo_enabled'] == true) {
        $doc = new DOMDocument();
        $doc->loadHTML($string);
        $images = $doc->getElementsByTagName('img');
        //$imgarray = array();
        foreach ($images as $image) {
            $url = $image->getAttribute('src');
            if(substr($url, 0, 8) != 'https://') {
                $image->setAttribute('data-original-src', $url);
                $camo = $this->proxy_url($url);
                $string = str_replace($image->getAttribute('src'), $camo, $string);
                unset($url, $camo);
            }
        }
    }
    return $string;
}

Answer 1

我认为问题出在您忘记修改您传递给调用saveHTML() 的方法的$string 变量，然后再返回它。按如下方式提交您的更改：

public function parse($string)
{
    if ($this->settings['camo_enabled'] == true)
    {
        $doc = new DOMDocument();
        $doc->loadHTML($string);
        $images = $doc->getElementsByTagName('img');

        foreach ($images as $image)
        {
            $url = $image->getAttribute('src');

            if (substr($url, 0, 8) != 'https://')
            {
                $image->setAttribute('data-original-src', $url);
                $camo = $this->proxy_url($url);
                $string = str_replace($image->getAttribute('src'), $camo, $string);
                unset($url, $camo);
            }
        }
    }

    $string = $doc->saveHTML(); 
    return $string;

    // The two lines above can also be simply rewritten as:
    // return $doc->saveHTML();
}

有关详细信息，请参阅official documentation。

附带说明，由于您正在对 $string 变量和已解析的 HTML 文件执行更改，请通过修改此行来坚持后者：

$string = str_replace($image->getAttribute('src'), $camo, $string);

进入：

$image->setAttribute('src', $camo);