Bugs found: missing CR and wrong replacement of &
-
Hello,
Very useful plugin for WordPress multisite.
However, I have found some little bugs.
1. When other plugins add some lines in robots.txt, they appear on the same line. Ex: Disallow: /wp-admin/Disallow xxx
The solution is to add a trailing CR in the option just before saving it.2. the & character is encoded to & That is not correct.
A solution is to not use esc_html() and esc_attr().Here are the patches to fix these bugs:
Index: core.php
===================================================================
— core.php (revision 148089)
+++ core.php (revision 148090)
@@ -29,14 +29,13 @@
*
* @since 1.0
* @uses get_option
– * @uses esc_attr
* @return string
*/
function cd_rdte_filter_robots($rv, $public)
{
$content = get_option(‘cd_rdte_content’);
if ($content) {
– $rv = esc_attr(strip_tags($content));
+ $rv = strip_tags($content);
}return $rv;
Index: options-page.php
===================================================================
— options-page.php (revision 148089)
+++ options-page.php (revision 148090)
@@ -123,7 +123,6 @@
* robots.txt field
*
* @since 1.0
– * @uses esc_html
* @uses add_settings_error
*/
public function cleanSetting($in)
@@ -138,7 +137,7 @@
);
}– return esc_html(strip_tags($in));
+ return strip_tags($in) . “\n”;
}/**
- The topic ‘Bugs found: missing CR and wrong replacement of &’ is closed to new replies.