Additional updates for processwire/processwire-issues#1467

Author: ryancramerdesign

Diff for: wire/core/WireTextTools.php

@@ -905,7 +905,7 @@ protected function ___wordAlternates($word, array $options) {
 	 *  - `has` (bool): Specify true to only return true or false if it has tags (default=false).
 	 * 	- `tagOpen` (string): The required opening tag character(s), default is '{'
 	 *	- `tagClose` (string): The required closing tag character(s), default is '}'
-	 * @return array|bool
+	 * @return array|bool Always returns array unless you specify the `has` option as true.
 	 * @since 3.0.126
 	 *
 	 */
@@ -918,6 +918,7 @@ public function findPlaceholders($str, array $options = array()) {
 		);
 
 		$options = array_merge($defaults, $options);
+		$str = (string) $str;
 		$tags = array();
 		$pos1 = strpos($str, $options['tagOpen']);
 

Diff for: wire/modules/Fieldtype/FieldtypeComments/FieldtypeComments.module

@@ -300,12 +300,12 @@ class FieldtypeComments extends FieldtypeMulti {
 		// assign code and subcode
 		if(!$comment->code || !$comment->subcode) {
 			$rand = new WireRandom();
-			if(!strlen($comment->code)) {
+			if(!strlen("$comment->code")) {
 				// code: visible to admin only
 				$code = $rand->alphanumeric(128);
 				$comment->code = $code;
 			}
-			if(!strlen($comment->subcode)) {
+			if(!strlen("$comment->subcode")) {
 				// subcode: may be visible to commenter
 				$subcode = $rand->alphanumeric(40);
 				$comment->subcode = $subcode;
@@ -2555,4 +2555,3 @@ class FieldtypeComments extends FieldtypeMulti {
 	}
 
 }
-

Diff for: wire/modules/Fieldtype/FieldtypeRepeater/InputfieldRepeater.module

@@ -263,7 +263,7 @@ class InputfieldRepeater extends Inputfield implements InputfieldItemList {
 
 		$fields = $this->wire()->fields;
 		$items = $this->attr('value');
-		$item = count($items) ? $items->first() : null;
+		$item = wireCount($items) ? $items->first() : null;
 
 		$templateId = $this->field ? (int) $this->field->get('template_id') : 0;
 		$template = $templateId ? $this->wire()->templates->get($templateId) : null;

Diff for: wire/modules/Inputfield/InputfieldPageTable/InputfieldPageTable.module

@@ -518,7 +518,7 @@ class InputfieldPageTable extends Inputfield {
 			$value = (string) $object;
 		}
 
-		$value = $this->wire()->sanitizer->entities(strip_tags($value)); 
+		$value = $this->wire()->sanitizer->entities(strip_tags("$value")); 
 		$value = nl2br($value); 
 
 		return $value; 

Diff for: wire/modules/Textformatter/TextformatterEntities.module

@@ -26,9 +26,9 @@ class TextformatterEntities extends Textformatter {
 	public function format(&$str) {
 		if($this->charset === '') $this->charset = $this->wire()->config->dbCharset;
 		if(stripos($this->charset, 'utf8') === 0) {
-			$str = htmlspecialchars($str, ENT_QUOTES, 'UTF-8');
+			$str = htmlspecialchars("$str", ENT_QUOTES, 'UTF-8');
 		} else {
-			$str = htmlspecialchars($str, ENT_QUOTES);
+			$str = htmlspecialchars("$str", ENT_QUOTES);
 		}
 	}
 }