- Issue created by @Nathan Tsai
- 🇨🇦Canada Nathan Tsai
We could probably just change:
if ($handler && isset($handler['data_function']) && function_exists($handler['data_function'])) {
to
if ($handler && isset($handler['name'], $handler['data_function']) && function_exists($handler['data_function'])) {
- First commit to issue fork.
- @sakthi_dev opened merge request.
- Status changed to Needs review
over 1 year ago 6:50am 19 April 2023 - 🇮🇳India sakthi_dev
Created a MR with the changes mentioned in #2. Please review.
- 🇺🇸United States caschbre
The issue is that $handler['data_function'] is returning FALSE. The output is not checked before attempting to set $data['handler'].
$data = call_user_func($handler['data_function'], $url); $data['handler'] = $handler['name']
Need to ensure that $data is an array before setting $data['handler'].
- Status changed to Needs work
over 1 year ago 6:58pm 18 May 2023 - First commit to issue fork.
- Merge request !16Issue#3355024: Setting data to array before setting to handler. → (Open) created by Unnamed author
- Status changed to Needs review
over 1 year ago 4:58pm 18 June 2023 - 🇮🇳India bharath-kondeti Hyderabad
Raised a new MR with changes https://git.drupalcode.org/project/video_embed_field/-/merge_requests/16
- First commit to issue fork.
- 🇺🇦Ukraine tibezh
I've updated the MR, just made the code style better.
Changes are still available at the https://git.drupalcode.org/project/video_embed_field/-/merge_requests/16... MR. - Status changed to RTBC
10 months ago 4:09pm 5 February 2024 - 🇨🇦Canada Nathan Tsai
Any updates on this issue? It's creating errors on the backend of one of our client's websites.