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.
Upgrade the cod_sponsors Feature to D7
Comment | File | Size | Author |
---|---|---|---|
#45 | upgrade-cod-sponsors-1150350-45.patch | 99.36 KB | DyanneNova |
#41 | upgrade-cod-sponsors-1150350-41.patch | 107.58 KB | DyanneNova |
#38 | upgrade-cod-sponsors-1150350-38.patch | 103.26 KB | DyanneNova |
#37 | upgrade-cod-sponsors-1150350-37.patch.diff | 192.52 KB | DyanneNova |
#32 | upgrade-cod-sponsors-1150350-32.patch | 36.27 KB | dsdeiz |
Comments
Comment #1
druroot CreditAttribution: druroot commentedThis patch rerolls a nearly identical clone of the sponsor content type, attached fields, roles, image styles, menu links, permissions, strongarm comment settings, views, and adds a context for block placement on the sponsors page. I believe this is pretty much everything that was originally included in the d6 version.
Comment #2
oadaeh CreditAttribution: oadaeh commentedThe patch applies cleanly, but I noticed a few minor inconsistencies while working on the BoF Scheduler. I will make a list and enumerate them here when I am done with my other work.
Comment #3
mlncn CreditAttribution: mlncn commentedUsing COD Sponsors standalone this patch worked for me. Still applies, maybe should be committed and later issues worked on in follow-up?
Comment #4
oadaeh CreditAttribution: oadaeh commentedAttached is a patch that makes some minor changes to @druroot's patch.
These are some of the changes made:
As an FYI, there is a Menu conflict with COD Session. They both add a menu item that adds as a dependency the 'main-menu' menu.
Comment #5
oadaeh CreditAttribution: oadaeh commentedComment #6
oadaeh CreditAttribution: oadaeh commentedTrying to get the patch right.
Comment #7
hosef CreditAttribution: hosef commentedThe patch no longer applies. Please re-roll the patch.
Comment #8
rocketeerbkw CreditAttribution: rocketeerbkw commentedPatch in #4 applies cleanly and, except for the main-menu conflict with COD Session, works great. What's the procedure for fixing the conflict? Just get rid of it?
Comment #9
mrconnerton CreditAttribution: mrconnerton commentedI basically took the patch from #1, applied it, then recreated the feature without the menu and menu links since the view supplies that.
For clarity as well, #4 applies to cod_sponsor_sales which should probably be its own issue
Comment #10
saltednut#9 does not apply cleanly - this is what I am seeing:
Comment #11
saltednutThis patch is a rebuild of the entire cod_sponsors feature with some minor architectural changes
Includes
Missing
-- Menu Items for the views under 'Conference management' See: #1441672: D7: Restore COD Base Conference admin menu links
It should apply using the latest version 7.x-1.x-dev - on my enviro it is showing up as overridden on the Features overview (/admin/structure/features) but there is nothing overridden when I inspect the actual Feature (/admin/structure/features/cod_sponsors) - this may have something to do with the cod_base dependency being overridden #1670284: cod_base feature has override on field permissions when enabled
Comment #12
gregglesCan you describe those?
Comment #13
saltednut-- This is per: #1658282: Evaluation of the Entityreferences Module vs References
-- Field permissions in D7 are much more granular than D6, so I had to rebuild those.
I suppose not including the menu items could be considered an architectural difference. However, the intent is to re-roll once #1670284: cod_base feature has override on field permissions when enabled is resolved.
Comment #14
twardnw CreditAttribution: twardnw commentedGetting a couple of errors when trying to enable the patched feature:
Looking at the patched feature, looks like some places reference field_logo and other field_sponsor_logo, also, I tried to re-export the feature and am coming back with missing image styles. Working to resolve these.
Comment #15
ezra-g CreditAttribution: ezra-g commentedMarking NW based on 14.
Comment #16
saltednutI believe there may be an issue with field_permissions going on here.
Submitted is a new patch that does not use field_permissions - lets see if we can get this one to apply and work upward towards including field_permissions.
Missing from this patch:
-- Granular field_permissions for sponsor content type
-- Menu links per #1670284: cod_base feature has override on field permissions when enabled
Comment #17
saltednutComment #18
twardnw CreditAttribution: twardnw commentedPatch applies clean, but I am seeing a conflict with COD Sponsorship Sales on permissions for "delete own sponsor content" and "edit own sponsor content".
Should we resolve those in COD Sponsors or in Sponsorship Sales?
Comment #19
gregglesI think sponsorship_sales is probably the best place to fix it.
Comment #20
twardnw CreditAttribution: twardnw commentedOk, removed those permission settings from the feature. It immediately shows overridden in admin/structure/features though, but inspecting the feature does not show what is being overridden :shrug: re-rolled patch attached.
Comment #21
saltednut#20 applies cleanly for me and no longer shows conflict with cod_sponsorship_sales
Comment #22
sheldonkreger CreditAttribution: sheldonkreger commentedPatch in #20 applies cleanly and conflict no longer shows with cod_sponsorship_sales.
Comment #23
sheldonkreger CreditAttribution: sheldonkreger commentedComment #24
dsdeiz CreditAttribution: dsdeiz commentedPatch applies cleanly on my end. But this still seems to use 6.x? It doesn't allow me to enable the feature on a fresh install.
Comment #25
dsdeiz CreditAttribution: dsdeiz commentedI've manually just changed
core = 6.x
tocore = 7.x
as this seems to be still an issue in features - #1014522: Upgrade existing features to D7. I'm not sure how it was done in cod_session or cod_base though. Patch attached.I'm still getting an "overridden" state too. Diff shows:
I can't seem to find where the menu "menu-conference-admin" is created.Ah looks like it's part of #1679940: Remove admin.module dependency.Comment #26
primerg CreditAttribution: primerg commentedtagging
Comment #27
primerg CreditAttribution: primerg commentedpatch tested using sandbox. I think #20 was already committed in the sandbox so the patch is in #20 and #25 is not working. Tried to reroll it and the only change is
When I enabled the module, the sponsor content type is empty.
Comment #28
dsdeiz CreditAttribution: dsdeiz commentedLooking at patches #20 and #16, #20 seems to not define fields. #16 mentioned fields in the patch. I'm guessing a file is missing for #16. Perhaps cod_sponsors.features.field.inc.
Comment #29
primerg CreditAttribution: primerg commentedreroll using the latest sandbox release
Comment #30
mjonesdinero CreditAttribution: mjonesdinero commentedapplies patch, and it applies cleanly..
also confirm that fields are present in the node/add/sponsor
Comment #31
dsdeiz CreditAttribution: dsdeiz commentedApplies cleanly on my end as well. I also see the fields when going to node/add/sponsor. The logos on the views though are not linked to the nodes. Also getting overridden state but I'm probably just missing some patches.
Comment #32
dsdeiz CreditAttribution: dsdeiz commentedSorry, missed the patch.
Comment #33
saltednut#32 applies cleanly to the sandbox, shows up as compatible with D7 and I'm able to enable the cod_sponsors Feature.
Cleanup attempts to disable the module when I enabled the Feature?
Features overview shows cod_base and cod_sponsors overridden after enabling but inspecting further shows no overrides.
Comment #34
uniqorn CreditAttribution: uniqorn commenteduntagging for da_drupalcon
Comment #35
sheldonkreger CreditAttribution: sheldonkreger commented#32 is clean and works as expected. I tested the views and blocks, too. When I add a sponsor logo to any sponsor node, it's displaying on the node, as well as in the Views. Plus, the image links back to the sponsor node.
I tried uploading some large images for sponsor logos. They are resized when they are displayed in blocks, but don't really fit correctly. On the sponsor node itself, the original image is shown. These are theme specific issues which will have to be configured based on where blocks are placed and how much space is in that region. So, I'm marking this as rtbc.
Comment #36
chaskype CreditAttribution: chaskype commentedbasic tests, it's working
Comment #37
DyanneNovaHere's a patch to update COD to da_drupalcon cod_sponsors + changes we've discussed at the Capital Camp sprint.
Comment #38
DyanneNovaOops, #37 included the DA folder. Here's the correct patch.
Comment #40
ezra-g CreditAttribution: ezra-g commentedI did a visual inspection of the patch:
A) Not sure why we have taxonomy_forums in here.
+ 'field' => 'taxonomy_forums:tid',
B) We've removed sponsorship levels but kept imagecache presets that refer to levels such as "bronze".
Let's name these according to the sizes of images they create and move them to cod_base.
Comment #41
DyanneNovaOK, I've removed all of the imagecache presets, and the weirdness with the taxonomy sort.
Comment #42
twardnw CreditAttribution: twardnw commentedPatch applies clean but I am immediately seeing overrides on the feature, here's the output of drush fd
I am able to add sponsorship levels and associate sponsors to them. When I visit the /sponsors page though, I am presented with a broken image, that might just be my dev environment. Although the view is set up to show the original image, perhaps we should change that to medium?
Comment #43
twardnw CreditAttribution: twardnw commentedAlso just got an error when attempting to go to admin/modules
PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'module' cannot be null: INSERT INTO {role_permission} (rid, permission, module) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => 1 [:db_insert_placeholder_1] => view field_logo [:db_insert_placeholder_2] => ) in user_role_grant_permissions() (line 3034 of /Users/twardnw/Sites/cod7.dev/modules/user/user.module).
After I disabled the cod_sponsors feature this error went away.
Comment #44
ezra-g CreditAttribution: ezra-g commentedIf we're removing image styles, I would expect that we would define equivalent ones in COD_Base so that they're available for use by cod_sponsors and other features.
Comment #45
DyanneNovaThis patch changes Sponsorship Level to a term reference.
Comment #46
japerryApplies great, and views and content looks right!
Comment #47
japerryCommited!
c3a5979655221411c1135fb37362c668948eecd8
Please open any new cod_Sponsors tickets as an individual issue instead of this big upgrade one.
Comment #48
gregglesNote that when you fix something the best Status is "fixed." This leaves it visible in the queue for two weeks at which point it gets automatically closed by an automated process. Leaving it in the queue makes it visible to people who may be looking for the same issue and helps prevent duplicate issues.
Also, congrats on becoming a co-maintainer, japerry and thanks for your work!