Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
At least ViewMode presumes content_node_field_instance exists. It needs a requirements check.
Comment | File | Size | Author |
---|---|---|---|
#12 | 2211611-12.patch | 2.34 KB | benjy |
#9 | interdiff.txt | 648 bytes | benjy |
#9 | 2211611-9.patch | 3.1 KB | benjy |
#7 | interdiff.txt | 3.01 KB | benjy |
#7 | 2211611-7.patch | 3.1 KB | benjy |
Comments
Comment #1
chx CreditAttribution: chx commentedComment #2
eliza411 CreditAttribution: eliza411 commentedTested with the latest pull (7cf6b33a5), and there's no longer an error.
Comment #3
chx CreditAttribution: chx commentedOK then reworded for the current problem and demoting.
Comment #4
ultimikeI think this is still an issue, as I just ran into it. Here's what I did:
1. Started with a brand-new D6 site, only the "standard" install. CCK ("Content" module) not enabled, confirmed that "content_node_field_instance" table doesn't exist in the database.
2. Added 2 nodes to the D6 site (one "story", one "page").
3. Ran the migration using this manifest: https://drupal.org/comment/8615887#comment-8615887
4. Received the following error:
I then enabled CCK ("Content" module only), and re-ran the migration - both nodes imported properly, with the following output:
-mike
Comment #5
ultimikeComment #6
benjy CreditAttribution: benjy commentedAdded a checkRequirements() to the ViewModeBase source class.
I also fixed up two tiny coding standard errors in the tests that use that source class while I was testing :)
Comment #7
benjy CreditAttribution: benjy commentedOK, turns out we shouldn't be using checkRequirements() anymore but source_provider. I've updated the two child classes to use source_provider and also fixed up UploadInstance which was still using checkRequirements().
Comment #9
benjy CreditAttribution: benjy commentedarghhh
Comment #10
chx CreditAttribution: chx commentedVery nice bugfix bouquet.
Comment #12
benjy CreditAttribution: benjy commentedRe-rolled.
Comment #13
chx CreditAttribution: chx commentedFinally the bot agrees with me :)
Comment #14
webchickCommitted and pushed to 8.x. Thanks!