The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi,
Our site name contained some special chars that made the google news xml invalid. So I made a small change that passes the site name through htmlentities. That might help also others since a single "&" char in the site name could make the xml invalid:
$content .= '<n:name>' . htmlentities(variable_get('site_name', 'Drupal')) . '</n:name>';
cheers
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | googlenews-n1110598-2.patch | 421 bytes | damienmckenna |
| #1 | escape-site-name-1110598-1.patch | 728 bytes | jaytennier |
Comments
Comment #1
jaytennier commentedThis is still a problem in the Drupal 7 version so rather than create a new issue I'm just bumping up the version number. Attached is a patch that fixes the issue in 7.x-1.4 and can easily be backported to 6.x. The only difference from the OP's solution is that I'm using
check_plain().Comment #2
damienmckennaDue to other changes going on in the module, this is even simpler now.
Comment #3
damienmckennaCommitted.
Comment #4
damienmckennaNeeds to be backported to D6.
Comment #5
damienmckennaComment #6
damienmckennaThe D6 version is no longer supported. Sorry folks.