Closed (outdated)
Project:
Serial Field
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Feb 2011 at 08:27 UTC
Updated:
25 Apr 2016 at 09:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
kirsh commentedPlease try the attached fix.
Note: update.php has to be run because this new version changes the schema.
Comment #2
geek-merlin@kirsh: can you roll a patch?
see: http://drupal.org/patch
ad-thanks-vance!
Comment #3
FranciscoLuz commentedPlease find attached the .patch files generated from thread #1.
The changes made by kirsh seem to have fixed the problem described on this issue ( column 'n.language' ) but now when enabling any module (admin/build/modules) I get this:
user warning: Unknown column 'i.type' in 'where clause' query: SELECT f.field_name FROM content_node_field f, content_node_field_instance i WHERE f.field_name = i.field_name AND f.type = 'serial' AND i.type = 'image' in ...\sites\all\modules\_cck\serial\serial.inc on line 51.
Are you folks able to give any light on this? :) Cheers
Comment #4
geek-merlin@kirsh, is it possible to separate the bugfix from additional patches you did?
that might brighten the life of the maintainer as well as that of helpful folks.
Comment #5
geek-merlinwell, just notices you *are* the maintainer.
Comment #6
geek-merlin@francisco: yes this *is* a bug, as of the schema it must be "i.type_name" (may not make sense, but schema sais it).
strange that you get that at module enable time (really any module?) because that line should only fire when a content type is renamed.
Comment #7
geek-merlinok fixed that (untested!) on top of kirsh's version from #1 and put it in my sandbox for all that have interest or further patches:
http://drupal.org/sandbox/axel.rutz/1154028
Comment #8
FranciscoLuz commented@axel.rutz #6
Well then it might be cache related. I mean, perhaps the error is not issued on every module enabling process but it gets printed out because it is cached or something. Just a guess!!!
When I get a chance I will re-test it.
Comment #9
zilverdistel commentedsubscribing
Whenever I clear my cache with drush, I get the following (twice):
Comment #10
geek-merlinit would really be great to understand under what conditions multilang (which module?) rewrites the sql.
my naive guess would that it checks if the node table is involved... but that is clearly not the case in serial.inc:49.
edit: here is a good explanation what happens. as we do not have a "db_rewrite_sql" in that function anymore i ask you, zilverdistel, to check if your source is the same as that in the link above. (if not you owe us a beer ;-)
Comment #11
zilverdistel commented@axel.rutz: I guess I owe you a beer after all. Removing the db_rewrite_sql call fixed the problem with
drush cc all.Has this been committed in dev allready?
Comment #12
mojzis commentedplease, please ... can we have a release with this solved ...
Comment #13
cyberwolf commentedSubscribing.
Comment #14
brutuscat commented#7 @axel.rutz Thanks, your fix seems to be working fine. It also fixed the cache issue. I applied it into my code with serial fields working and it continues to work without issues. I haven't tried renaming the field though.
Comment #15
zilverdistel commentedI think this can be marked RTBC ...
Comment #16
colanAdding tag.
Comment #17
colanI don't see a patch here with the most recent fix. Is this even still a problem? It looks like at least some of the code is already in the dev version. Can someone confirm? I no longer have D6 running.
If it's still a problem, please provide the individual fix in patch format that applies to the current dev branch.
Comment #18
colanAs nobody's complained, I'll assume that this is no longer an issue.
Comment #21
tusik commentedThis unfortunately continues to be an issue. After updating core to 6.30, I am getting the same error message for i.types 'blog' and 'panel'
Comment #22
tusik commentedComment #23
mustanggb commented