From 85853136db9a564b1d979900534756fad0775a8f Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Mon, 24 Jul 2017 19:28:04 +0200
Subject: [PATCH] Enable rule which was previously turned off.

The reason the rule was turned off was that it gave problems with concatenation across multiple lines.
Turns out there is a property which can be set to selectively turn the rule off when the string is multiline.

Problem solved ;-)

Ref: https://github.com/squizlabs/PHP_CodeSniffer/wiki/Customisable-Sniff-Properties#genericstringsunnecessarystringconcat
---
 Yoast/ruleset.xml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Yoast/ruleset.xml b/Yoast/ruleset.xml
index 920a0128..48b1596b 100644
--- a/Yoast/ruleset.xml
+++ b/Yoast/ruleset.xml
@@ -82,8 +82,11 @@
 	</rule>
 
 	<!-- Application memory use! -->
-    <!-- Temporarily disabled to discuss, see https://github.com/Yoast/yoastcs/issues/4 R. -->
-	<!--<rule ref="Generic.Strings.UnnecessaryStringConcat" />-->
+	<rule ref="Generic.Strings.UnnecessaryStringConcat">
+		<properties>
+			<property name="allowMultiline" value="true" />
+		</properties>
+	</rule>
 
 	<!-- Error prevention: Make sure the condition in a inline if declaration is bracketed -->
 	<rule ref="Squiz.ControlStructures.InlineIfDeclaration" />