♻️ Rename flag to better show actual effect

This flag only causes parameters with empty strings to be skipped, not all empty() values.
dokuwiki · Jun 29, 2018 · 8fd9398 · 8fd9398
1 parent fdbbfed
commit 8fd9398
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/inc/common.php b/inc/common.php
@@ -356,16 +356,16 @@ function buildURLparams($params, $sep = '&amp;') {
  *
  * @author Andreas Gohr
  *
- * @param array $params    array with (attribute name-attribute value) pairs
- * @param bool  $skipempty skip empty string values?
+ * @param array $params           array with (attribute name-attribute value) pairs
+ * @param bool  $skipEmptyStrings skip empty string values?
  * @return string
  */
-function buildAttributes($params, $skipempty = false) {
+function buildAttributes($params, $skipEmptyStrings = false) {
     $url   = '';
     $white = false;
     foreach($params as $key => $val) {
         if($key{0} == '_') continue;
-        if($val === '' && $skipempty) continue;
+        if($val === '' && $skipEmptyStrings) continue;
         if($white) $url .= ' ';
 
         $url .= $key.'="';