comma separated lists of tags not created - it is create some lines with the same name, but different context:

<meta name="keywords" value="first" />
<meta name="keywords" value="ev" />
<meta name="keywords" value="vehicles" />

But it is need to be comma separated:

<meta name="keywords" value="first, ev, vehicles" />

I find this string near at line 127 in int_meta.module:

foreach ($tag->values as $value) {set_html_head('<meta name="' . check_plain($name) . '" value="' . check_plain($value) . '" />');}

but it is need to be something like this:

foreach ($tag->values as $value) {
			   $vals = "$vals, $value";	
			}
			drupal_set_html_head('<meta name="' . check_plain($name) . '" content="' . check_plain($vals) . '" />');

NOTE: for META tags elements "value" is not good - element "content" is using for META tag

CommentFileSizeAuthor
#6 Edit vocabulary.png12.49 KBArshad Vayani
#6 Page.png10.48 KBArshad Vayani
#4 1.jpg41.19 KBdaspit

Comments

daspit’s picture

Assigned: daspit » Unassigned
daspit’s picture

Version: 6.x-1.0-beta » 6.x-1.0-beta2

Thank to author for fast update. "content" element looks much better then "value".
But comma separated lists is not created with new version of int_meta.

fractile81’s picture

Do you have the "Combine" checkbox selected for the field you want to have as a comma separated list? Also, to combine different fields (e.g. you want to combine a node.type and term.*.name fields together), the fields have to have the same name AND the "Combine" checkbox selected.

If it's still not working quite right, can you take a picture of your Integrated Metatags configuration page and attach it here? I believe this is just a configuration problem, but am willing to investigate it. Or, just explain your configuration and I'll see if I can reproduce it. Before the BETA2 came out, I had tested this and I had it working on my data.

daspit’s picture

Status: Active » Fixed
StatusFileSize
new41.19 KB

Thank you, now all work fine.
It was bug of my theme: not all columns are shown (very long width for "name" input field - I attach the screenshot to comment).
Can you change the position of "combine" and "delete" columns to the left of "name" field?

fractile81’s picture

Component: Code » User interface
Category: bug » feature
Status: Fixed » Active

Updated this to be a feature request for later.

Arshad Vayani’s picture

Component: User interface » Code
Category: feature » bug
Status: Active » Fixed
StatusFileSize
new10.48 KB
new12.49 KB

i am still getting the same error?? i dont understand how to fix it here are the screenshots of my settings

fractile81’s picture

Component: Code » User interface
Assigned: Unassigned » fractile81
Category: bug » feature

You have a fixed-width theme. There's a "Combine" checkbox to the right of the "Name" field, which your theme is cutting off. To fix this, you can either change your admin theme to be a non-fixed-width theme to see the checkbox, or update the database directly. The SQL would look something like:

UPDATE int_meta_fields SET combine = 1 WHERE name = 'keywords'

My plan is to move the checkbox to the left of the name field to help people with fixed-width themes.

Arshad Vayani’s picture

ohh i got it now Thanks ;)

fractile81’s picture

Status: Fixed » Active

Updated the status since this is now a Feature Request.

fractile81’s picture

Status: Active » Fixed

I've rearranged the checkboxes now, so let me know if they helped out!

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.