Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
hi
i see thats my google meta tag is not between the head so google cant found it
thanks
Comment | File | Size | Author |
---|---|---|---|
#9 | site-verify-842440-9.patch | 619 bytes | David_Rothstein |
#6 | site-verify-842440-4.patch | 1.1 KB | David_Rothstein |
Comments
Comment #1
Dave ReidCan you please copy/paste the output you're getting? Or link to the website so I can see?
Comment #2
eule CreditAttribution: eule commentedhi dave i try the new 7.x-1.0-rc2
if i add the meta tag nothing is visible..cache cleaned
the other think is i try to delete the meta tag under "Operations"
this calls this error
Recoverable fatal error: Argument 2 passed to db_query() must be an array, string given, called in x/sites/all/modules/site_verify/site_verify.admin.inc on line 248 and defined in db_query() (line 2166 of x/includes/database/database.inc).
i upped the meta file and it works
Comment #3
David_Rothstein CreditAttribution: David_Rothstein commentedLooks to me like meta tags aren't getting added to the page at all, due to this code:
The API for this function is different in Drupal 7: http://api.drupal.org/api/function/drupal_add_html_head/7
Comment #4
Dave ReidConfirmed and on it.
Comment #5
Dave ReidThanks all for reporting and confirming! This is now fixed in CVS: http://drupal.org/cvs?commit=413190
Comment #6
David_Rothstein CreditAttribution: David_Rothstein commentedOops, I should have assigned this to myself then... I was already working on a patch also :( Didn't think you'd respond so quickly, I guess!
In any case, here it is. Hopefully we didn't duplicate any work.
Comment #7
David_Rothstein CreditAttribution: David_Rothstein commentedHa, almost the same exact patch, oh well :)
Thanks for the incredibly speedy response.
Comment #8
Dave ReidHeh, we're both quick on the draw. Your patch had some improvements to formatting and the query, so I committed those as well and gave you credit. :)
http://drupal.org/cvs?commit=413194
Comment #9
David_Rothstein CreditAttribution: David_Rothstein commentedCool, thanks. But I just ran CVS up to test it out and I think in the process of merging the two a bug was reintroduced (so now if you have more than one meta tag it doesn't work).
Followup patch attached...
Comment #10
Dave ReidSigh, and that's what I get for not testing it the second time...
http://drupal.org/cvs?commit=413196