The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I get this warning by following the steps below:
Warning: require_once(/Users/alex/Sites/d7/head/modules/taxonomy/taxonomy.feeds.inc): failed to open stream: No such file or directory in menu_execute_active_handler() (line 460 of /Users/alex/Sites/d7/head/includes/menu.inc).
Steps to reproduce:
- Clean d7 install
- Create a new story with a tag
- browse to /taxonomy/term/1/edit
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | 632888_5_combined.patch | 2.12 KB | bleen |
| #8 | 632888_4.patch | 839 bytes | bleen |
| #8 | 632888_4test.patch | 1.44 KB | bleen |
| #6 | 632888_3.patch | 2.18 KB | bleen |
| #3 | 632888.patch | 839 bytes | bleen |
Comments
Comment #1
bleen commentedI assume that taxonomy.feeds.inc is needed to create RSS feeds but the file is missing from HEAD. Assuming that the file is no longer needed (which I doubt), this patch fixes the error.
Comment #2
mr.baileysI was able to reproduce this bug, although the problem occurs on /taxonomy/term/1/feed, not on /taxonomy/term/1/edit...
While the patch in #1 fixes the issue, it goes a bit too far and removes the feed menu item entirely. It looks like
taxonomy_term_feedwas moved totaxonomy.pages.inc, so it's enough to just change the file name.@bleen18: can you re-roll the patch with just that change?
Comment #3
bleen commentedre-rolled according to mr.baileys suggestion.
Comment #4
catchPatch is good. Ideally we'd also add to one of the tests in taxonomy.test to do a drupalGet("taxonomy/term/$tid/feed"); - which should fail without the patch applied.
Comment #5
bleen commented@catch ... agreed. This may be a good opportunity for me to figure out how to actually create tests... I'll play. In the mean time can we mark RTBC?
Comment #6
bleen commentedre-rolled patch to include a simpletest ... it's the first test I've added and it seemed ... well... too simple so please make sure I did it correctly when reviewing. Thanks!
Comment #7
catchThat looks completely fine. Can you do two things?
1. Post a patch with just the test so we can watch it fail tests. This is optional but verifies the test actually gets affected by the patch.
2. Post that same patch, but without extra whitespace on the empty lines.
Then this is rtbc.
Comment #8
bleen commentedSlowly but surely I'm getting all the nuances down
Comment #10
catchAwesome. Now we just need the combined patch from #6 without the trailing whitespace so it can be committed in one pass.
Comment #11
bleen commentedcool beans
Comment #12
catchComment #13
dries commentedCommitted to CVS HEAD. Thanks!