r92417 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r92416‎ | r92417 | r92418 >
Date:22:45, 17 July 2011
Author:yaron
Status:deferred
Tags:
Comment:
Partial revert of r92410 - removed HTML-escaping; this is the wrong place for it
Modified paths:
  • /trunk/extensions/SemanticForms/includes/SF_FormPrinter.php (modified) (history)

Diff [purge]

Index: trunk/extensions/SemanticForms/includes/SF_FormPrinter.php
@@ -807,7 +807,7 @@
808808 $field_query_val = $template_instance_query_values[$field_name];
809809 }
810810 // Escaping of quotes is necessary.
811 - $field_query_val = htmlentities( $field_query_val );
 811+ $field_query_val = str_replace( '"', '"', $field_query_val );
812812 if ( $form_submitted || ( ! empty( $field_query_val ) && ! is_array( $field_query_val ) ) ) {
813813 $cur_value = $field_query_val;
814814 }

Follow-up revisions

RevisionCommit summaryAuthorDate
r92420Follow-up to r92410 and r92417 - this is the wrong place for any sort of HTML...yaron00:26, 18 July 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r92410Fixed some formatting, and added HTML-escaping of values from query string, s...yaron20:42, 17 July 2011