From 0a53cdd88fabd6fa7c8331208dea0acbdc940ad8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marcos=20Garci=CC=81a=20de=20La=20Fuente?= Date: Sun, 20 Mar 2016 13:33:51 +0100 Subject: [PATCH] Corrected non-static usages of Form::multiselectarray function --- htdocs/adherents/card.php | 4 ++-- htdocs/contact/card.php | 4 ++-- htdocs/core/class/html.form.class.php | 2 +- htdocs/core/class/html.formactions.class.php | 2 +- htdocs/product/card.php | 4 ++-- htdocs/public/test/test_forms.php | 2 +- htdocs/societe/soc.php | 8 ++++---- htdocs/user/card.php | 4 ++-- 8 files changed, 15 insertions(+), 15 deletions(-) diff --git a/htdocs/adherents/card.php b/htdocs/adherents/card.php index 71ae89ddc19..92b1bcbdd70 100644 --- a/htdocs/adherents/card.php +++ b/htdocs/adherents/card.php @@ -913,7 +913,7 @@ else { print '' . fieldLabel('Categories', 'memcars') . ''; $cate_arbo = $form->select_all_categories(Categorie::TYPE_MEMBER, null, 'parent', null, null, 1); - print $form->multiselectarray('memcats', $cate_arbo, GETPOST('memcats', 'array'), null, null, null, null, '100%'); + print Form::multiselectarray('memcats', $cate_arbo, GETPOST('memcats', 'array'), null, null, null, null, '100%'); print ""; } @@ -1173,7 +1173,7 @@ else foreach ($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray('memcats', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); + print Form::multiselectarray('memcats', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); print ""; } diff --git a/htdocs/contact/card.php b/htdocs/contact/card.php index fc6196370d3..a2b55c3c034 100644 --- a/htdocs/contact/card.php +++ b/htdocs/contact/card.php @@ -662,7 +662,7 @@ else if (! empty($conf->categorie->enabled) && ! empty($user->rights->categorie->lire)) { print '' . fieldLabel( 'Categories', 'contcats' ) . ''; $cate_arbo = $form->select_all_categories( Categorie::TYPE_CONTACT, null, 'parent', null, null, 1 ); - print $form->multiselectarray( 'contcats', $cate_arbo, GETPOST( 'contcats', 'array' ), null, null, null, + print Form::multiselectarray( 'contcats', $cate_arbo, GETPOST( 'contcats', 'array' ), null, null, null, null, '90%' ); print ""; } @@ -922,7 +922,7 @@ else foreach ($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray( 'contcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%' ); + print Form::multiselectarray( 'contcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%' ); print ""; } diff --git a/htdocs/core/class/html.form.class.php b/htdocs/core/class/html.form.class.php index 8e88fd31c99..6e8af102616 100644 --- a/htdocs/core/class/html.form.class.php +++ b/htdocs/core/class/html.form.class.php @@ -4846,7 +4846,7 @@ class Form * @return string HTML multiselect string * @see selectarray */ - static function multiselectarray($htmlname, $array, $selected=array(), $key_in_label=0, $value_as_key=0, $morecss='', $translate=0, $width=0, $moreattrib='',$elemtype='') + public static function multiselectarray($htmlname, array $array, array $selected = array(), $key_in_label=0, $value_as_key=0, $morecss='', $translate=0, $width=0, $moreattrib='',$elemtype='') { global $conf, $langs; diff --git a/htdocs/core/class/html.formactions.class.php b/htdocs/core/class/html.formactions.class.php index cf46547b9d5..8fe32b0dd70 100644 --- a/htdocs/core/class/html.formactions.class.php +++ b/htdocs/core/class/html.formactions.class.php @@ -273,7 +273,7 @@ class FormActions if (! empty($multiselect)) { if(!is_array($selected) && !empty($selected)) $selected = explode(',', $selected); - print $form->multiselectarray($htmlname, $arraylist, $selected, 0, 0, 'centpercent', 0, 0); + print Form::multiselectarray($htmlname, $arraylist, $selected, 0, 0, 'centpercent', 0, 0); } else { diff --git a/htdocs/product/card.php b/htdocs/product/card.php index b94d775db40..76ec3e8e057 100644 --- a/htdocs/product/card.php +++ b/htdocs/product/card.php @@ -1040,7 +1040,7 @@ else // Categories print ''.$langs->trans("Categories").''; $cate_arbo = $form->select_all_categories(Categorie::TYPE_PRODUCT, '', 'parent', 64, 0, 1); - print $form->multiselectarray('categories', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); + print Form::multiselectarray('categories', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); print ""; } @@ -1343,7 +1343,7 @@ else foreach($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray('categories', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); + print Form::multiselectarray('categories', $cate_arbo, $arrayselected, '', 0, '', 0, '100%'); print ""; } diff --git a/htdocs/public/test/test_forms.php b/htdocs/public/test/test_forms.php index 08086110cf2..732afa2c331 100644 --- a/htdocs/public/test/test_forms.php +++ b/htdocs/public/test/test_forms.php @@ -86,7 +86,7 @@ print '

'."\n"; print "Test 5c: a multiselect
\n"; $array=array(1=>'Value 1',2=>'Value 2',3=>'Value 3'); $arrayselected=array(1,3); -print $form->multiselectarray('testmulti', $array, $arrayselected, '', 0, '', 0, 250); +print Form::multiselectarray('testmulti', $array, $arrayselected, '', 0, '', 0, 250); llxFooter(); diff --git a/htdocs/societe/soc.php b/htdocs/societe/soc.php index 09fee411888..22d3c6294e1 100644 --- a/htdocs/societe/soc.php +++ b/htdocs/societe/soc.php @@ -1242,7 +1242,7 @@ else if ($object->prospect || $object->client) { print '' . fieldLabel('CustomersCategoriesShort', 'custcats') . ''; $cate_arbo = $form->select_all_categories(Categorie::TYPE_CUSTOMER, null, 'parent', null, null, 1); - print $form->multiselectarray('custcats', $cate_arbo, GETPOST('custcats', 'array'), null, null, null, + print Form::multiselectarray('custcats', $cate_arbo, GETPOST('custcats', 'array'), null, null, null, null, "90%"); print ""; } @@ -1251,7 +1251,7 @@ else if ($object->fournisseur) { print '' . fieldLabel('SuppliersCategoriesShort', 'suppcats') . ''; $cate_arbo = $form->select_all_categories(Categorie::TYPE_SUPPLIER, null, 'parent', null, null, 1); - print $form->multiselectarray('suppcats', $cate_arbo, GETPOST('suppcats', 'array'), null, null, null, + print Form::multiselectarray('suppcats', $cate_arbo, GETPOST('suppcats', 'array'), null, null, null, null, "90%"); print ""; } @@ -1800,7 +1800,7 @@ else foreach ($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray('custcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%'); + print Form::multiselectarray('custcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%'); print ""; } @@ -1814,7 +1814,7 @@ else foreach ($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray('suppcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%'); + print Form::multiselectarray('suppcats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%'); print ""; } } diff --git a/htdocs/user/card.php b/htdocs/user/card.php index 194974bbc40..a027395a712 100644 --- a/htdocs/user/card.php +++ b/htdocs/user/card.php @@ -1087,7 +1087,7 @@ if (($action == 'create') || ($action == 'adduserldap')) { print '' . fieldLabel( 'Categories', 'usercats' ) . ''; $cate_arbo = $form->select_all_categories( Categorie::TYPE_USER, null, 'parent', null, null, 1 ); - print $form->multiselectarray( 'usercats', $cate_arbo, GETPOST( 'usercats', 'array' ), null, null, null, + print Form::multiselectarray( 'usercats', $cate_arbo, GETPOST( 'usercats', 'array' ), null, null, null, null, '90%' ); print ""; } @@ -2231,7 +2231,7 @@ else foreach ($cats as $cat) { $arrayselected[] = $cat->id; } - print $form->multiselectarray( 'usercats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%' ); + print Form::multiselectarray( 'usercats', $cate_arbo, $arrayselected, '', 0, '', 0, '90%' ); print ""; }