chore: mark global statements for DI refactoring

lib/Horde/Form/Renderer.php

@@ -72,6 +72,7 @@ public static function Horde_Form_Renderer($params = [])
 
     public function __construct($params = [])
     {
+        // TODO: refactor globals to DI
         global $registry;
         if (isset($registry) && is_a($registry, 'Registry')) {
             /**
@@ -516,6 +517,7 @@ public function _renderVarInputEnd($form, $var, $vars)
 
         /* Display any help for the field. */
         if ($var->hasHelp()) {
+            // TODO: refactor globals to DI
             global $registry;
             if (isset($registry) && $registry instanceof Horde_Registry) {
                 $link = Horde_Help::link($GLOBALS['registry']->getApp(), $var->getHelp());

lib/Horde/Form/Type.php

@@ -1420,6 +1420,7 @@ private function _getUpload($vars, $var)
             return;
         }
 
+        // TODO: refactor globals to DI
         global $session;
 
         $varname = $var->getVarName();

src/V3/ImageVariable.php

@@ -238,6 +238,7 @@ private function _getUpload($vars)
             return;
         }
 
+        // TODO: refactor globals to DI
         global $session;
 
         $varname = $this->getVarName();
@@ -349,6 +350,7 @@ public function getImage($vars, ...$args)
             self::Deprecated('Warning: The second ($var) parameter in getImage() is deprecated/ignored');
         }
 
+        // TODO: refactor globals to DI
         global $session;
 
         $this->_getUpload($vars);