Magento\Config\Model\Config\Structure\AbstractElement::getPath() ignores a custom `config_path` value

dmitry_fedyuk · December 12, 2017, 11:59am

It is wrong:

magento/magento2/blob/2.2.1/app/code/Magento/Config/Model/Config/Structure/AbstractElement.php#L197-L219


/**
 * Retrieve config path for given id
 *
 * @param string $fieldId
 * @param string $fieldPrefix
 * @return string
 */
protected function _getPath($fieldId, $fieldPrefix = '')
{
    $path = isset($this->_data['path']) ? $this->_data['path'] : '';
    return $path . '/' . $fieldPrefix . $fieldId;
}


/**
 * Retrieve element config path
 *
 * @param string $fieldPrefix
 * @return string
 */
public function getPath($fieldPrefix = '')
{
    return $this->_getPath($this->getId(), $fieldPrefix);
}

dmitry_fedyuk · December 12, 2017, 12:11pm

A workaround:

github.com

mage2pro/core/blob/3.4.7/Config/lib/structure.php#L133-L143


/**
* 2017-12-12
* 1) "@uses \Magento\Config\Model\Config\Structure\AbstractElement::getPath()
* ignores a custom `config_path` value": https://mage2.pro/t/5148
* 2) @uses df_config_field() returns a flyweight: https://en.wikipedia.org/wiki/Flyweight_pattern
* @return string
*/
function df_config_field_path() {
	$f = df_config_field(); /** @var Field $f */
	return $f->getConfigPath() ?: $f->getPath();
}