๐Ÿ‡ฎ๐Ÿ‡ณIndia @anup.singh

Account created on 27 December 2011, over 13 years ago
  • Senior Software Engineer at Syngentaย  โ€ฆ
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

Hi @sarwan_verma,

Thanks for the quick patch, I have made some modification and also removed some extra spacing.

Attaching the updated patch, also raise the MR.

Thanks
Anup

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ changed the visibility of the branch 3458435-deprecated-strpos-passing to active.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ changed the visibility of the branch 3458435-deprecated-strpos-passing to hidden.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

Patch #3 fails if you have a plain text area (In my case Revision log message)

So updated patch to work with that.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

Worked for me after

composer require laminas/laminas-escaper 
๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

Created a merge request using the suggestion from #9,
But I added the call on the new update hook too, as it is failing when upgrading core to D10

Note: this is still not the final fix for this issue, but just a workaround to unblock the broken updb

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anup.singh

anup.singh โ†’ made their first commit to this issueโ€™s fork.

Production build 0.71.5 2024