HTMLForm: Use PHP 5.6 splat operator directly instead of ObjectFactory
authorKunal Mehta <legoktm@member.fsf.org>
Tue, 4 Jun 2019 19:45:38 +0000 (15:45 -0400)
committerKunal Mehta <legoktm@member.fsf.org>
Wed, 5 Jun 2019 03:49:20 +0000 (23:49 -0400)
Change-Id: I3fddf30f1b0fc2cce79f652cb71efba119439a9f

includes/htmlform/HTMLForm.php

index acf64db..d071478 100644 (file)
@@ -21,8 +21,6 @@
  * @file
  */
 
-use Wikimedia\ObjectFactory;
-
 /**
  * Object handling generic submission, CSRF protection, layout and
  * other logic for UI forms. in a reusable manner.
@@ -304,12 +302,11 @@ class HTMLForm extends ContextSource {
 
                switch ( $displayFormat ) {
                        case 'vform':
-                               return ObjectFactory::constructClassInstance( VFormHTMLForm::class, $arguments );
+                               return new VFormHTMLForm( ...$arguments );
                        case 'ooui':
-                               return ObjectFactory::constructClassInstance( OOUIHTMLForm::class, $arguments );
+                               return new OOUIHTMLForm( ...$arguments );
                        default:
-                               /** @var HTMLForm $form */
-                               $form = ObjectFactory::constructClassInstance( self::class, $arguments );
+                               $form = new self( ...$arguments );
                                $form->setDisplayFormat( $displayFormat );
                                return $form;
                }