Closed (fixed)
Project:
Fences - Semantic field markup and classes
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 May 2012 at 14:02 UTC
Updated:
8 Feb 2014 at 14:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #0.0
sylus commentedMod
Comment #1
Sborsody commentedHah, I was *just* doing the same thing! I get undefined index errors on lines 30 and 32. I added fences to the info file and was testing installation. There are some features with content types and fields installed too. I think it has something to do with the features being defined before fences was installed.
Comment #2
sylus commentedI am using features as well so could very well be the problem.
Comment #3
johnalbinUpdating title
Comment #4
sylus commentedI should mention I did manage to load fences during the installation but only when enabling it manually in the final install task. Obviously not a perfect solution but a temporary workaround.
Comment #5
johnalbinSo, fences implements hook_module_implements_alter so that it's fences_theme() runs after field_theme() because it needs to copy some data from the theme registry details specified in the "field" hook which gets added by field_theme().
The "undefined index" means that the theme registry is empty for the "field" hook when fences_theme() is run. Can you add some debugging code to see if field_theme() is run before fences_theme()?
Comment #6
adamdicarlo commented@JohnAlbin,
Have you seen Jose Reyero's comment on the doc page for hook_module_implements_alter()?
Issues he linked: #496170: module_implements() cache can be polluted by module_invoke_all() being called (in)directly prior to full bootstrap completion, #1267966: Pathauto patterns are not showing up (see comment #87).
Like the OP, I'm getting this Fences PHP notice a bajillion times when enabling the module in an install profile via the .info file.
Comment #7
roysegall commentedGuys, i ran in to this problem as well. In one of the instllation profile i'm activating the module. The problem is that there is no implementation by any hook and we get this error.
I attached the patch i did and i didn't get any notices during the installation and the module work as it should be(the field get the right fence).
Enjoy ;
Comment #8
sylus commentedWorks for me thanks @RoySegall for the fix!
Comment #9
roysegall commentedi dont have a rtbc but i think it should be fine for that.
Comment #10
johnalbinOk. Since the fix for this is easy and only affects the not-often-called theme registry build, I'm fine with adding this patch.
HOWEVER, NOTE THESE 2 THINGS:
Fixed! http://drupalcode.org/project/fences.git/commitdiff/9c6d7d6
Thanks, Roy! http://drupal.org/node/1284600/committers
Comment #11.0
(not verified) commentedMod
Comment #12
minoroffense commentedI'm currently working on an install profile as this issue is still present in the latest stable version of Fences. Any chance of a 1.1 to fix the bug? It's been a while since a release was made regardless.