Closed (fixed)
Project:
Nodewords: D6 Meta Tags
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Jan 2010 at 16:54 UTC
Updated:
9 Nov 2011 at 14:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
summit commentedHi,
Also on vocabulary level : admin/content/taxonomy/edit/vocabulary/[vid] the altered Meta tag description and keywords are not saved..
Greetings, Martijn
Comment #2
summit commentedHi,
I tried to debug this myself..but am just an amateur...
It seems that the taxonomy tags are not correctly processed through
function nodewords_taxonomy($op, $type, $object = NULL)
It seems that $type = 'term' and not NODEWORDS_TYPE_TERM = 6.
the same with vocabulary.
So I changed the function amateuristic to:
Hopefully helps this for you to get the error.
Greetings,
Martijn
Comment #3
avpadernoThe cause of the error was the use of a constant that doesn't match the value passed to
hook_taxonomy().I changed the code, and now the hook is implemented in this way:
Thanks for your report, and the help.
Comment #4
avpadernoI have marked #673086: Meta tags KEYWORDS and DESCRIPTION are not saved as duplicate of this report.
Comment #5
gafir777 commentedHello,
I'm having the same issue with 6.x-1.11. Has the fix been implemented in the version 6.x-1.11 ?
Thanks,
Comment #6
summit commentedHi, no in the dev version, the maintainer said that after releasing a version, code-changes are only submitted in the future .dev releases. I hope I tell this right, because I am not the maintainer of this great module!
But the current dev version is also having problems, see: http://drupal.org/node/673624 ..hopefully small issue for the maintainer.
Greetings,
Martijn
Comment #7
avpaderno@Summit: What you report is correct; an official release cannot be changed, after it is released.
I was once used to change the referring version to make clear where the issue was fixed, but I was told that it could confuse people who would not know which version had the issue.
Comment #8
avpadernoI have marked #679852: Meta tags are not saved for taxomony terms as duplicate of this report.
Comment #10
jenlampton@kiamlaluno -
When you fix a bug, can you please provide a patch against the current release for the rest of us to use? It's much safer to run a patched version of 6.x-1.11 rather than a dev version on a live site.
Thanks,
Jen
Comment #11
tharrison commentedThanks for making the patch file, Jen.
Comment #12
Anonymous (not verified) commentedI have marked #763204: Not saved metatags for taxonomy term in database as duplicate of this report.
To reply of why I didn't attach a patch for version 6.x-1.11, I work on the development snapshot; that is the reason I cannot attach a patch for an official release (which is different from the development snapshot).
Comment #13
Rameez commented@jenlampton How to apply this patch? please say in simple steps? Thanks
Comment #14
venusrising commentedDoes anyone know what version to use that works? OR if there is something else that will do the same thing that is stable. This is a real mess to follow.
Comment #15
venusrising commentedWill #10 work to fix this version until another stable release happens?
Comment #16
Anonymous (not verified) commentedSee any official release created after this issue has been set as fixed.
Comment #17
venusrising commentedwhat does this mean? Is there any version of nodewords that works without issue i.e all parts of the module work like saving other pages etc? It is hard to tell when people are making patchs but then they do not apply. Thanks for your hard work.
Comment #18
Anonymous (not verified) commentedThis issue report has been marked as fixed on January 2, 2010 − 05:14 AM GMT. This means that about 7 hours later the script running on Drupal.org created a new package for the development snapshot which contained the fixed code; the first official release created after that development snapshot contains also the fixed code because the official releases are always created from a development snapshot.
If there is anything else that is not clear, please let me know.
Comment #19
venusrising commentedSo just to be clear, am I understanding this correctly?
1. when you do a fix it is applied to the dev version OR
2. Is the fixes applied to the stable versions & then we would just overwrite the stable version and run update php.
The reason I ask is I tried to go to dev from 6x 1.11 and I got errors.
Thank so much for your hard work.
Comment #20
venusrising commentedDo you mean January 12? I do not see any releases for January 2nd
Comment #21
avpadernoIt's January 2, 2010; see comment #3. About 7 hours later, a new development snapshot has been created. The official release is the first one created after that date; an official release can be created even 7 days later (or even more), as an official release is usually created when more than one issue is fixed.
You understood correctly when you say that the code is first fixed in a development snapshot; that is how Drupal modules hosted on Drupal.org are developed.
Comment #22
Anonymous (not verified) commentedComment #23
tomsm commentedsubscribing
Comment #24
onetrader commentedsubscribing
Comment #26
venusrising commentedThis NEW version of 6.x-1.11 Does not save any info on other pages, What can be done to fix this, we have a production site and this has really damaged a great deal of work and we are trying to dig out of it. Any input would be appreciated.
Comment #27
kollo-dherbois commented"Will #10 work to fix this version until another stable release happens?"
It fixed problem with saving taxonomy term`s metas for me.
Comment #28
kollo-dherbois commented"This NEW version of 6.x-1.11 Does not save any info on other pages, What can be done to fix this, we have a production site and this has really damaged a great deal of work and we are trying to dig out of it. Any input would be appreciated."
Why you call 6.x-1.11 new version? It`s release date is 2009-Dec-31. I do not experience such problem with 6.x-1.11 as you mention. I might guess there some wrong settings you applied - try different options and see what happens. Nodewords UI is quite entangled, so sometimes you would get things you not expected.
Comment #29
damienmckennaMarked the following as duplicates:
Comment #30
mcurry commentedSub :D
Comment #31
damienmckennaRerolled jenlampton's patch from #10 above for the latest 6.x-1.x branch.
Comment #33
damienmckennaRe-running the last patch against the correct code version.
Comment #34
damienmckenna#31: nodewords-n672096-31.patch queued for re-testing.
Comment #35
dave reidRe-applied the commits from this issue, so this should be good now.
Comment #36
u210ks commentedsubscribing
Comment #38
koppie commentedI can confirm that the latest dev version (6.x-1.x-dev 2011-9-13) works!
Comment #39
halloffame commentedThe latest dev version ought to solve my 'not-saving' problem but the fieldset still goes under save button. Weight should be fixed as well.