<commit_before><?php
namespace Df\Config\Block\System\Config\Form;
use Magento\Config\Block\System\Config\Form\Fieldset as _Fieldset;
use Magento\Framework\Data\Form\Element\AbstractElement;
// 2015-12-13
// Хитрая идея, которая уже давно пришла мне в голову: наследуясь от модифицируемого класса,
// мы получаем возможность вызывать методы с областью доступа protected у переменной $subject.
class FieldsetPlugin extends _Fieldset {
	/**
	 * 2016-01-01
	 * Потрясающая техника, которую я изобрёл только что.
	 */
	public function __construct() {}

	/**
	 * 2015-12-21
	 * Цель перекрытия — устранения дефекта:
	 * «Magento 2 backend incorrectly renders the nested fieldsets:
	 * adds nested TR tags with the same id».
	 * https://mage2.pro/t/330
	 *
	 * Этот дефект приводит к неработоспособности условия <depends> для элемента:
	 * т.е. видимость элемента перестаёт зависеть от другой опции.
	 *
	 * @see \Magento\Config\Block\System\Config\Form\Fieldset::render()
	 * @param _Fieldset|Fieldset $subject
	 * @param \Closure $proceed
	 * @param AbstractElement $element
	 * @return \Magento\Framework\Option\ArrayInterface|mixed
	 */
	public function aroundRender(_Fieldset $subject, \Closure $proceed, AbstractElement $element) {
		$subject->setElement($element);
		$html = $subject->_getHeaderHtml($element);
		foreach ($element->getElements() as $field) {
			if (
				$field instanceof \Magento\Framework\Data\Form\Element\Fieldset
				// 2015-12-21
				// Вот в этой добавке и заключается суть модифицации.
				&& !$field instanceof \Df\Framework\Data\Form\Element\Fieldset
			) {
				$html .= '<tr id="row_' . $field->getHtmlId() . '"><td colspan="4">' . $field->toHtml() . '</td></tr>';
			}
			else {
				$html .= $field->toHtml();
			}
		}
		$html .= $subject->_getFooterHtml($element);
		return $html;
	}
}

<commit_msg>no message<commit_after><?php
namespace Df\Config\Block\System\Config\Form;
use Magento\Config\Block\System\Config\Form\Fieldset as _Fieldset;
use Magento\Framework\Data\Form\Element\AbstractElement;
// 2015-12-13
// Хитрая идея, которая уже давно пришла мне в голову: наследуясь от модифицируемого класса,
// мы получаем возможность вызывать методы с областью доступа protected у переменной $subject.
class FieldsetPlugin extends _Fieldset {
	/**
	 * 2016-01-01
	 * Потрясающая техника, которую я изобрёл только что.
	 */
	public function __construct() {}

	/**
	 * 2015-12-21
	 * Цель перекрытия — устранения дефекта:
	 * «Magento 2 backend incorrectly renders the nested fieldsets:
	 * adds nested TR tags with the same id».
	 * https://mage2.pro/t/330
	 *
	 * Этот дефект приводит к неработоспособности условия <depends> для элемента:
	 * т.е. видимость элемента перестаёт зависеть от другой опции.
	 *
	 * @see \Magento\Config\Block\System\Config\Form\Fieldset::render()
	 * @param _Fieldset|Fieldset $subject
	 * @param \Closure $proceed
	 * @param AbstractElement $element
	 * @return string
	 */
	public function aroundRender(_Fieldset $subject, \Closure $proceed, AbstractElement $element) {
		/** @var string $result */
		/**
		 * 2016-01-01
		 * Потомки @see \Magento\Config\Block\System\Config\Form\Fieldset могли перекрыть метод
		 * @see \Magento\Config\Block\System\Config\Form\Fieldset::render().
		 * Пример: @see \Magento\Config\Block\System\Config\Form\Fieldset\Modules\DisableOutput::render()
		 * Поэтому в случае с классом-потомком неправильно не вызывать метод render().
		 */
		if (get_class($subject) !== _Fieldset::class) {
			$result = $proceed($element);
		}
		else {
			$subject->setElement($element);
			$result = $subject->_getHeaderHtml($element);
			foreach ($element->getElements() as $field) {
				if (
					$field instanceof \Magento\Framework\Data\Form\Element\Fieldset
					// 2015-12-21
					// Вот в этой добавке и заключается суть модифицации.
					&& !$field instanceof \Df\Framework\Data\Form\Element\Fieldset
				) {
					$result .= df_tag('tr', ['id' => 'row_' . $field->getHtmlId(),
						df_tag('td', ['colspan' => 4])], $field->toHtml()
					);
				}
				else {
					$result .= $field->toHtml();
				}
			}
			$result .= $subject->_getFooterHtml($element);
		}
		return $result;
	}
}


{'fraction_non_alphanumeric': 0.13644006725918809, 'fraction_numerical': 0.020177756425654576, 'mean_word_length': 3.6577181208053693, 'pattern_counts': {'":': 0, '<': 11, '<?xml version=': 0, '>': 25, 'https://': 2, 'lorem ipsum': 0, 'www.': 0, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 0, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '4257749', 'n_tokens_mistral': 1652, 'n_tokens_neox': 1562, 'n_words': 335}