After #2854819: Replace deprecated url method many AJAX forms fail, like page creation wizard in page manager UI. An Url object is assigned where a string is expected.

CommentFileSizeAuthor
#2 ctools-url_to_string_forms-3033651-2.patch4.88 KBmanuel.adan

Comments

manuel.adan created an issue. See original summary.

manuel.adan’s picture

manuel.adan
Location 🌌
commented
Assigned: manuel.adan » Unassigned
Status: Active » Needs review
StatusFileSize
new ctools-url_to_string_forms-3033651-2.patch4.88 KB
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

Thanks @manuel.adan for catching that. Reading the deprecation notice doesn't explicitly indicate the return type change, would be nice to have a __toString() method on that object, IMO

  • joelpittet committed 5e774aa on 8.x-3.x authored by manuel.adan 
    Issue #3033651 by manuel.adan: AJAX forms broken after URL refactoring,...
manuel.adan’s picture

manuel.adan
Location 🌌
commented

The Url::__toString() was in 8.0 but deprecated soon after #2416971: Remove Url::__toString()

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented

do we need reroll against latest 4.x dev or is it fixed there already?

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented

Apart from that Page Manager UI still not working. Short AJAX clock popping up near the button and nothing happens.