Site and Alternate Locales broken in v5.2

[jpmaga]

So this code in v5.1 was working alright:

$tags = new SeoProTags();
$tags->setContext($this->entry->toAugmentedCollection());
$meta = $tags->metaData();

giving the following output:

    "site": {
      "handle": "en",
      "name": "English",
      "lang": "en",
      "locale": "en-US",
      "short_locale": "en",
      "url": "/en",
      "permalink": "http://sandbox.test/en",
      "direction": "ltr",
      "attributes": []
    },
    "alternate_locales": [
      {
        "site": {
          "handle": "other",
          "name": "Other",
          "lang": "ot",
          "locale": "ot-HER",
          "short_locale": "ot",
          "url": "/other",
          "permalink": "http://sandbox.test/other",
          "direction": "ltr",
          "attributes": []
        },
        "url": "http://sandbox.test/other"
      }
    ],

But when i updated to 5.2, this is what i get now:

    "site": {},
    "alternate_locales": [
      {
        "site": {},
        "url": "http://kit.test/other"
      }
    ],

If i revert back to 5.1 then it works again. No code was changed when trying this.

Notes: I have disabled statamic routing, I am using laravel's Route::fallback to handle requests, and this piece of code is part of the controller.

[jasonvarga]

The tag class is intended to be used as a tag. Not really arbitrarily in a controller.

But you can work around this by adding the following lines.

$tags = new SeoProTags();
$tags->setContext($this->entry->toAugmentedCollection());
$meta = $tags->metaData();
+$meta['site'] = $meta['site']->toAugmentedArray();
+$meta['alternate_locales'] = collect($meta['alternate_locales'])->map(function ($alt) {
+  return array_merge($alt, ['site' => $alt['site']->toAugmentedArray()]);
+})->all();

[jpmaga]

Yeah, this is a bit of a workaround as mentioned in #248 and #264. But thanks for sharing that solution for the time being.