Needs review
Project:
Domain Blocks
Version:
7.x-2.x-dev
Component:
Documentation
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
16 Mar 2011 at 13:42 UTC
Updated:
17 Apr 2013 at 21:52 UTC
Jump to comment: Most recent file
Comments
Comment #1
nonsieThis patch should fix it
Comment #2
choy commentedthe patch doesn't work - domain_block still cause site crash to anonymous users
error message with schema: domain_blocks.domain_id is type int but its default 0 is PHP type string
Comment #3
skizzo commentedOnly after installing the "schema" module (just out of curiosity), I see the reported warning at admin/structure/schema (not in status report)
domain_blocks.domain_id is type int but its default 0 is PHP type stringApplying the patch to 7.x-2.0 the warning disappears but I get the following
Comment #4
agentrickardThat is a schema module bug.
This patch would also need an update hook.
Comment #5
clearbrook commentedHey guys. I was only looking at the 7.x-3.0-alpha1 branch, so I did not see what you were looking at here. nonsie pointed me to you guys. I will be banging on this, and I am as raw as it gets, but I do understand the need for an update hook quite clearly now. And there are likely other problems I will run into if I try to do anything other than an Alpha1 to Alpha2 update with as much as is in the mix already on this branch.
BTW, my tentative attempt is on:
http://drupal.org/node/1964638
I am going to look at what you guys are doing. It might shed some light on it for me.
(and Alpha1 to Alpha2 on the 3.x branch is not a complete fix, either, but all I am going to tackle on my first attempt)
Comment #6
clearbrook commentedHmmm. In looking at your patch, I see that what I am dealing with is already dealt with at this point in this branch. And the problem you have here is already corrected on 3.x Alpha1 (although I don't know about the update hook) So should I consider an update hook to cover from Alpha1 to Alpha2 with my problem (which is very similar to yours) on 3.x good enough?
Comment #7
nonsieUpdate in this context means converting the field for those sites already using Domain Blocks. The patch as it is only helps those who have not yet installed the module.
Comment #8
nonsieHere's a patch for 7.x-2.x-dev that includes the original path from #1 as well as an update for those sites that already have this module installed.
Also fixes some minor style/instruction issues.