The path of a URI with an authority must start with a slash "/" or be empty about embed

[oscarotero]

The problem is undefined is a valid value for the href attribute, this is an issue in the website side.

A solution would be assume the "undefined" value as an empty value, but this opens the door to other values that must be considered empty like "null", "nil", etc. I don't know if make such assumptions is a good idea.

[grandeljay]

Hmmm... Are the values null, nil etc also valid? My assumption is that there was some issue with JavaScript and the value has undefined which surely isn't intentional. I mean, why would anybody want undefined, etc here? I think it would be best if you handle it.

In this case, this small change should fix the issue.

From:

Embed/src/Detectors/Languages.php

Line 20 in 9cb2dff

if (!$language || !$href) {

To:

if (!$language || !$href || 'undefined' === $href) {

Full code:

<?php

declare(strict_types=1);

namespace Embed\Detectors;

class Languages extends Detector
{
    /**
     * @return \Psr\Http\Message\UriInterface[]
     */
    public function detect(): array
    {
        $document  = $this->extractor->getDocument();
        $languages = array();

        foreach ($document->select('.//link[@hreflang]')->nodes() as $node) {
            $language = $node->getAttribute('hreflang');
            $href     = $node->getAttribute('href');

            if (!$language || !$href || 'undefined' === $href) {
                continue;
            }

            $languages[$language] = $this->extractor->resolveUri($href);
        }

        return $languages;
    }
}

Alternatively is there something I can do to handle this case? I really don't know what the best solution would be here.

[oscarotero]

In this case, I'd create a helper, so it can be used in other places. Something like:

function isEmpty($value) {
    return empty($value) || $value === "undefined";
}

If you want to work on a PR, I'll review and merge it.