minor #3555 Wrap variables in {} for safer interpolation (ifdattic)

weaverryan · weaverryan · commit f4bb017d99b2 · 2014-02-16T13:47:12.000-06:00
This PR was merged into the 2.3 branch. Discussion ---------- Wrap variables in {} for safer interpolation | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | 2.3 | Fixed tickets | Commits ------- 4046476 Replace double quotes with single quotes 0f7a189 Change interpolation to concatenation d3839ce Wrap variables in {} for safer interpolation
diff --git a/components/http_foundation/sessions.rst b/components/http_foundation/sessions.rst
@@ -28,7 +28,7 @@ Quick example::
 
     // retrieve messages
     foreach ($session->getFlashBag()->get('notice', array()) as $message) {
-        echo "<div class='flash-notice'>$message</div>";
+        echo '<div class="flash-notice">'.$message.'</div>';
     }
 
 .. note::
@@ -308,18 +308,18 @@ Simple, display one type of message::
 
     // display warnings
     foreach ($session->getFlashBag()->get('warning', array()) as $message) {
-        echo "<div class='flash-warning'>$message</div>";
+        echo '<div class="flash-warning">'.$message.'</div>';
     }
 
     // display errors
     foreach ($session->getFlashBag()->get('error', array()) as $message) {
-        echo "<div class='flash-error'>$message</div>";
+        echo '<div class="flash-error">'.$message.'</div>';
     }
 
 Compact method to process display all flashes at once::
 
     foreach ($session->getFlashBag()->all() as $type => $messages) {
         foreach ($messages as $message) {
-            echo "<div class='flash-$type'>$message</div>\n";
+            echo '<div class="flash-'.$type.'">'.$message.'</div>';
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ Quick example::`
`28`	`28`
`29`	`29`	`// retrieve messages`
`30`	`30`	`foreach ($session->getFlashBag()->get('notice', array()) as $message) {`
`31`		`- echo "<div class='flash-notice'>$message</div>";`
	`31`	`+ echo '<div class="flash-notice">'.$message.'</div>';`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`.. note::`
`@@ -308,18 +308,18 @@ Simple, display one type of message::`
`308`	`308`
`309`	`309`	`// display warnings`
`310`	`310`	`foreach ($session->getFlashBag()->get('warning', array()) as $message) {`
`311`		`- echo "<div class='flash-warning'>$message</div>";`
	`311`	`+ echo '<div class="flash-warning">'.$message.'</div>';`
`312`	`312`	`}`
`313`	`313`
`314`	`314`	`// display errors`
`315`	`315`	`foreach ($session->getFlashBag()->get('error', array()) as $message) {`
`316`		`- echo "<div class='flash-error'>$message</div>";`
	`316`	`+ echo '<div class="flash-error">'.$message.'</div>';`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`Compact method to process display all flashes at once::`
`320`	`320`
`321`	`321`	`foreach ($session->getFlashBag()->all() as $type => $messages) {`
`322`	`322`	`foreach ($messages as $message) {`
`323`		`- echo "<div class='flash-$type'>$message</div>\n";`
	`323`	`+ echo '<div class="flash-'.$type.'">'.$message.'</div>';`
`324`	`324`	`}`
`325`	`325`	`}`