-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHP notice Undefined index: module in feeds_system_info_alter() #13
Comments
Hm, I'm unable to reproduce this on the 1.x branch... but pushing my PR anyway. Perhaps it will help someone else. |
I haven't tested it, but I believe @jenlampton's PR would prevent the notices appearing. However, it doesn't fix the root problem, which is that Here's one of the array keys returned by
No So I think this issue needs some more work (by someone who knows more about how Feeds should work than I do). |
@BWPanda Drupal 7 uses |
Here's some docs on converting feeds plugins for Backdrop: It looks like maybe all that is needed is adding a module key to the plugins? I bet ctools did that automatically, but we can do it explicitly. |
I just ran into this again. It happened immediately when I first installed feeds module. |
…revent PHP notice.
I've filed a new PR that adds the missing module key to all feeds plugins in this module: #13 |
I'm getting flooded with these errors, which I assume is the same problem - just different line number.
|
@stpaultim have you tested the PR? I just applied that to a new site and it resolved the issue for me. |
@jenlampton I just replaced the feeds module with your branch and I'm no longer flooded with errors, but I am getting a single error.
I will try to do a little more testing later tonight. |
Do you have any feeds modules enabled? Itay be coming from that one.
…On Sat, Mar 19, 2022, 8:42 PM Tim Erickson ***@***.***> wrote:
@jenlampton <https://github.com/jenlampton> I just replaced the feeds
module with your branch and I'm not longer flooded with errors, but I am
getting a single error.
Notice: Undefined index: module in feeds_system_info_alter() (line 896 of
/var/www/triplo/all-modules/simplo/src/modules/contrib/feeds/feeds.module).
—
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADBER7AXW2YEBHTYRERRQDVA2ND3ANCNFSM4FSBLXEQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I'm seeing my dblog flodded with the following PHP notices:
The following code...
...should be replaced as follows:
PR: #13
The text was updated successfully, but these errors were encountered: