Posted by Summit on January 1, 2010 at 4:54pm
16 followers
| Project: | Nodewords: D6 Meta Tags |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | v6.x-1.12 blocker |
Issue Summary
Hi, I try now to manually alter my Meta Tags on my terms.
I do not get a warning or error, but the Meta Tags information is not saved...
A very strange thing is also that the Meta Tabs link is situated under the [SAVE] button. Could that have to do with it? (See attached picture)
Thanks for going into this.
Greetings,
Maritjn
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| metatags_taxonomy.jpg | 136 KB | Ignored: Check issue status. | None | None |
Comments
#1
Hi,
Also on vocabulary level : admin/content/taxonomy/edit/vocabulary/[vid] the altered Meta tag description and keywords are not saved..
Greetings, Martijn
#2
Hi,
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:
function nodewords_taxonomy($op, $type, $object = NULL) {
if ($type == NODEWORDS_TYPE_TERM || $type == 'term') {
if ($type == 'term') { $type = NODEWORDS_TYPE_TERM; }
$id = $object['tid'];
}
elseif ($type == NODEWORDS_TYPE_VOCABULARY || $type == 'vocabulary') {
if ($type == 'vocabulary') { $type = NODEWORDS_TYPE_VOCABULARY; }
$id = $object['vid'];
}
else {
return;
}
switch ($op) {
case 'delete':
nodewords_delete_tags($type, $id);
break;
case 'insert':
case 'update':
if (isset($object['nodewords'])) {
nodewords_save_tags($type, $id, $object['nodewords'], TRUE);
}
break;
}
}
Hopefully helps this for you to get the error.
Greetings,
Martijn
#3
The 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:
<?php
function nodewords_taxonomy($op, $type, $object = NULL) {
if (isset($object)) {
switch ($type) {
case 'term':
$type = NODEWORDS_TYPE_TERM;
$id = $object['tid'];
break;
case 'vocabulary':
$type = NODEWORDS_TYPE_VOCABULARY;
$id = $object['vid'];
break;
default:
return;
}
switch ($op) {
case 'delete':
nodewords_delete_tags($type, $id);
break;
case 'insert':
case 'update':
if (isset($object['nodewords'])) {
nodewords_save_tags($type, $id, $object['nodewords'], TRUE);
}
break;
}
}
}
?>
Thanks for your report, and the help.
#4
I have marked #673086: Meta tags KEYWORDS and DESCRIPTION are not saved as duplicate of this report.
#5
Hello,
I'm having the same issue with 6.x-1.11. Has the fix been implemented in the version 6.x-1.11 ?
Thanks,
#6
Hi, 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
#7
@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.
#8
I have marked #679852: Meta tags are not saved for taxomony terms as duplicate of this report.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.
#10
@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
#11
Thanks for making the patch file, Jen.
#12
I 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).
#13
@jenlampton How to apply this patch? please say in simple steps? Thanks
#14
Does 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.
#15
Will #10 work to fix this version until another stable release happens?
#16
See any official release created after this issue has been set as fixed.
#17
what 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.
#18
This 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.
#19
So 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.
#20
Do you mean January 12? I do not see any releases for January 2nd
#21
It'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.
#22
#23
subscribing
#24
subscribing
#25
Automatically closed -- issue fixed for 2 weeks with no activity.
#26
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.
#27
"Will #10 work to fix this version until another stable release happens?"
It fixed problem with saving taxonomy term`s metas for me.
#28
"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.
#29
Marked the following as duplicates:
#30
Sub :D
#31
Rerolled jenlampton's patch from #10 above for the latest 6.x-1.x branch.
#32
The last submitted patch, nodewords-n672096-31.patch, failed testing.
#33
Re-running the last patch against the correct code version.
#34
#31: nodewords-n672096-31.patch queued for re-testing.
#35
Re-applied the commits from this issue, so this should be good now.
#36
subscribing
#37
Automatically closed -- issue fixed for 2 weeks with no activity.
#38
I can confirm that the latest dev version (6.x-1.x-dev 2011-9-13) works!
#39
The latest dev version ought to solve my 'not-saving' problem but the fieldset still goes under save button. Weight should be fixed as well.