songlist_getFieldForm()

Complexity

Conditions	44
Paths	> 20000

Size

Total Lines	229
Code Lines	175

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	function songlist_getFieldForm(&$field, $action = false)
	{
		if ( $action === false ) {
			$action = $_SERVER['PHP_SELF'];
		}
		
		xoops_loadLanguage('forms', 'songlist');
		
		$title = $field->isNew() ? sprintf(_FRM_SONGLIST_FIELDS_ADD, _FRM_SONGLIST_FIELDS_FIELD) : sprintf(_FRM_SONGLIST_FIELDS_EDIT, _FRM_SONGLIST_FIELDS_FIELD);
	
		xoops_load('XoopsFormLoader');
	
		$form = new XoopsThemeForm($title, 'form', $action, 'post', true);
	
		$form->addElement(new XoopsFormText(_FRM_SONGLIST_FIELDS_TITLE, 'field_title', 35, 255, $field->getVar('field_title', 'e')));
		$form->addElement(new XoopsFormTextArea(_FRM_SONGLIST_FIELDS_DESCRIPTION, 'field_description', $field->getVar('field_description', 'e')));
	
		if (!$field->isNew()) {
			$fieldcid = $field->getVar('cids');
		} else {
			$fieldcid = array(1=>0);
		}
		$category_handler = xoops_getmodulehandler('category', 'songlist');
		$cat_select = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_CATEGORY, 'cids', $fieldcid, 7, true);
		$cat_select->addOption(0, _FRM_SONGLIST_FIELDS_DEFAULT);
		foreach($category_handler->getObjects(NULL, true) as $cid => $category)

The method getObjects() does not exist on XoopsObjectHandler. It seems like you code against a sub-type of said class. However, the method does not exist in XoopsRankHandler or XoUserHandler. Are you sure you never get one of those?

			$cat_select->addOption($cid, $category->getVar('name'));
		$form->addElement($cat_select);
		$form->addElement(new XoopsFormText(_FRM_SONGLIST_FIELDS_WEIGHT, 'field_weight', 10, 10, $field->getVar('field_weight', 'e')));
		if ($field->getVar('field_config') || $field->isNew()) {
			if (!$field->isNew()) {
				$form->addElement(new XoopsFormLabel(_FRM_SONGLIST_FIELDS_NAME, $field->getVar('field_name')));
				$form->addElement(new XoopsFormHidden('id', $field->getVar('field_id')));
			} else {
				$form->addElement(new XoopsFormText(_FRM_SONGLIST_FIELDS_NAME, 'field_name', 35, 255, $field->getVar('field_name', 'e')));
			}
	
			//autotext and theme left out of this one as fields of that type should never be changed (valid assumption, I think)
			$fieldtypes = array(
				'checkbox' => _FRM_SONGLIST_FIELDS_CHECKBOX,
				'date' => _FRM_SONGLIST_FIELDS_DATE,
				'datetime' => _FRM_SONGLIST_FIELDS_DATETIME,
				'longdate' => _FRM_SONGLIST_FIELDS_LONGDATE,
				'group' => _FRM_SONGLIST_FIELDS_GROUP,
				'group_multi' => _FRM_SONGLIST_FIELDS_GROUPMULTI,
				'language' => _FRM_SONGLIST_FIELDS_LANGUAGE,
				'radio' => _FRM_SONGLIST_FIELDS_RADIO,
				'select' => _FRM_SONGLIST_FIELDS_SELECT,
				'select_multi' => _FRM_SONGLIST_FIELDS_SELECTMULTI,
				'textarea' => _FRM_SONGLIST_FIELDS_TEXTAREA,
				'dhtml' => _FRM_SONGLIST_FIELDS_DHTMLTEXTAREA,
				'textbox' => _FRM_SONGLIST_FIELDS_TEXTBOX,
				'timezone' => _FRM_SONGLIST_FIELDS_TIMEZONE,
				'yesno' => _FRM_SONGLIST_FIELDS_YESNO);
	
			$element_select = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_TYPE, 'field_type', $field->getVar('field_type', 'e'));
			$element_select->addOptionArray($fieldtypes);
	
			$form->addElement($element_select);
	
			switch ($field->getVar('field_type')) {
				case "textbox":
					$valuetypes = array(
						XOBJ_DTYPE_ARRAY            => _FRM_SONGLIST_FIELDS_ARRAY,
						XOBJ_DTYPE_EMAIL            => _FRM_SONGLIST_FIELDS_EMAIL,
						XOBJ_DTYPE_INT              => _FRM_SONGLIST_FIELDS_INT,
						XOBJ_DTYPE_FLOAT            => _FRM_SONGLIST_FIELDS_FLOAT,
						XOBJ_DTYPE_DECIMAL          => _FRM_SONGLIST_FIELDS_DECIMAL,
						XOBJ_DTYPE_TXTAREA          => _FRM_SONGLIST_FIELDS_TXTAREA,
						XOBJ_DTYPE_TXTBOX           => _FRM_SONGLIST_FIELDS_TXTBOX,
						XOBJ_DTYPE_URL              => _FRM_SONGLIST_FIELDS_URL,
						XOBJ_DTYPE_OTHER    		=> _FRM_SONGLIST_FIELDS_OTHER);
	
					$type_select = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_VALUETYPE, 'field_valuetype', $field->getVar('field_valuetype', 'e'));
					$type_select->addOptionArray($valuetypes);
					$form->addElement($type_select);
					break;
	
				case "select":
				case "radio":
					$valuetypes = array(
						XOBJ_DTYPE_ARRAY            => _FRM_SONGLIST_FIELDS_ARRAY,
						XOBJ_DTYPE_EMAIL            => _FRM_SONGLIST_FIELDS_EMAIL,
						XOBJ_DTYPE_INT              => _FRM_SONGLIST_FIELDS_INT,
						XOBJ_DTYPE_FLOAT            => _FRM_SONGLIST_FIELDS_FLOAT,
						XOBJ_DTYPE_DECIMAL          => _FRM_SONGLIST_FIELDS_DECIMAL,
						XOBJ_DTYPE_TXTAREA          => _FRM_SONGLIST_FIELDS_TXTAREA,
						XOBJ_DTYPE_TXTBOX           => _FRM_SONGLIST_FIELDS_TXTBOX,
						XOBJ_DTYPE_URL              => _FRM_SONGLIST_FIELDS_URL,
						XOBJ_DTYPE_OTHER            => _FRM_SONGLIST_FIELDS_OTHER);
	
					$type_select = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_VALUETYPE, 'field_valuetype', $field->getVar('field_valuetype', 'e'));
					$type_select->addOptionArray($valuetypes);
					$form->addElement($type_select);
					break;
			}
	
			//$form->addElement(new XoopsFormRadioYN(_FRM_SONGLIST_FIELDS_NOTNULL, 'field_notnull', $field->getVar('field_notnull', 'e') ));
	
			if ($field->getVar('field_type') == "select" || $field->getVar('field_type') == "select_multi" || $field->getVar('field_type') == "radio" || $field->getVar('field_type') == "checkbox") {
				$options = $field->getVar('field_options');
				if (count($options) > 0) {
					$remove_options = new XoopsFormCheckBox(_FRM_SONGLIST_FIELDS_REMOVEOPTIONS, 'removeOptions');
					$remove_options->columns = 3;
					asort($options);
					foreach (array_keys($options) as $key) {
						$options[$key] .= "[{$key}]";
					}
					$remove_options->addOptionArray($options);
					$form->addElement($remove_options);
				}
	
				$option_text = "<table  cellspacing='1'><tr><td width='20%'>" . _FRM_SONGLIST_FIELDS_KEY . "</td><td>" . _FRM_SONGLIST_FIELDS_VALUE . "</td></tr>";
				for ($i = 0; $i < 3; $i++) {
					$option_text .= "<tr><td><input type='text' name='addOption[{$i}][key]' id='addOption[{$i}][key]' size='15' /></td><td><input type='text' name='addOption[{$i}][value]' id='addOption[{$i}][value]' size='35' /></td></tr>";
					$option_text .= "<tr height='3px'><td colspan='2'> </td></tr>";
				}
				$option_text .= "</table>";
				$form->addElement(new XoopsFormLabel(_FRM_SONGLIST_FIELDS_ADDOPTION, $option_text) );
			}
		}
	
		if ($field->getVar('field_edit')) {
			switch ($field->getVar('field_type')) {
				case "textbox":
				case "textarea":
				case "dhtml":
					$form->addElement(new XoopsFormText(_FRM_SONGLIST_FIELDS_MAXLENGTH, 'field_maxlength', 35, 35, $field->getVar('field_maxlength', 'e')));
					$form->addElement(new XoopsFormTextArea(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
	
				case "checkbox":
				case "select_multi":
					$def_value = $field->getVar('field_default', 'e') != null ? unserialize($field->getVar('field_default', 'n')) : null;
					$element = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $def_value, 8, true);
					$options = $field->getVar('field_options');
					asort($options);
					// If options do not include an empty element, then add a blank option to prevent any default selection
					if (!in_array('', array_keys($options))) {
						$element->addOption('', _NONE);
					}
					$element->addOptionArray($options);
					$form->addElement($element);
					break;
	
				case "select":
				case "radio":
					$def_value = $field->getVar('field_default', 'e') != null ? $field->getVar('field_default') : null;
					$element = new XoopsFormSelect(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $def_value);
					$options = $field->getVar('field_options');
					asort($options);
					// If options do not include an empty element, then add a blank option to prevent any default selection
					if (!in_array('', array_keys($options))) {
						$element->addOption('', _NONE);
					}
					$element->addOptionArray($options);
					$form->addElement($element);
					break;
	
				case "date":
					$form->addElement(new XoopsFormTextDateSelect(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', 15, $field->getVar('field_default', 'e')));
					break;
	
				case "longdate":
					$form->addElement(new XoopsFormTextDateSelect(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', 15, strtotime($field->getVar('field_default', 'e'))));
					break;
	
				case "datetime":
					$form->addElement(new XoopsFormDateTime(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', 15, $field->getVar('field_default', 'e')));
					break;
	
				case "yesno":
					$form->addElement(new XoopsFormRadioYN(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
	
				case "timezone":
					$form->addElement(new XoopsFormSelectTimezone(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
	
				case "language":
					$form->addElement(new XoopsFormSelectLang(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
	
				case "group":
					$form->addElement(new XoopsFormSelectGroup(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', true, $field->getVar('field_default', 'e')));
					break;
	
				case "group_multi":
					$form->addElement(new XoopsFormSelectGroup(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', true, $field->getVar('field_default', 'e'), 5, true));
					break;
	
				case "theme":
					$form->addElement(new XoopsFormSelectTheme(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
	
				case "autotext":
					$form->addElement(new XoopsFormTextArea(_FRM_SONGLIST_FIELDS_DEFAULT, 'field_default', $field->getVar('field_default', 'e')));
					break;
			}
		}
	
		$groupperm_handler = xoops_gethandler('groupperm');
		$searchable_types = array(
			'textbox',
			'select',
			'radio',
			'yesno',
			'date',
			'datetime',
			'timezone',
			'language');
		if (in_array($field->getVar('field_type'), $searchable_types)) {
			$search_groups = $groupperm_handler->getGroupIds('songlist_search', $field->getVar('field_id'), $GLOBALS['songlistModule']->getVar('mid'));

The method getGroupIds() does not exist on XoopsObjectHandler. It seems like you code against a sub-type of XoopsObjectHandler such as XoopsGroupPermHandler or XoopsPersistableObjectHandler.

			$form->addElement(new XoopsFormSelectGroup(_FRM_SONGLIST_FIELDS_PROF_SEARCH, 'songlist_search', true, $search_groups, 5, true) );
		}
		if ($field->getVar('field_edit') || $field->isNew()) {
			if (!$field->isNew()) {
				//Load groups
				$editable_groups = $groupperm_handler->getGroupIds('songlist_edit', $field->getVar('field_id'), $GLOBALS['songlistModule']->getVar('mid'));
			} else {
				$editable_groups = array();
			}
			$form->addElement(new XoopsFormSelectGroup(_FRM_SONGLIST_FIELDS_PROF_EDITABLE, 'songlist_edit', false, $editable_groups, 5, true));
			$form->addElement($steps_select);

The variable $steps_select seems to be never defined.

		}
		$form->addElement(new XoopsFormHidden('op', 'save') );
		$form->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
	
		return $form;
	}
?>

It is not recommended to use PHP's closing tag ?> in files other than templates.