mirror of
https://github.com/Karaka-Management/oms-ItemManagement.git
synced 2026-02-16 04:08:43 +00:00
fix todos/move to todo list
This commit is contained in:
parent
b514502e4e
commit
07f6e51962
|
|
@ -15,7 +15,6 @@ declare(strict_types=1);
|
||||||
namespace Modules\ItemManagement\Controller;
|
namespace Modules\ItemManagement\Controller;
|
||||||
|
|
||||||
use phpOMS\Module\ModuleAbstract;
|
use phpOMS\Module\ModuleAbstract;
|
||||||
use phpOMS\Module\WebInterface;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* ItemManagement controller class.
|
* ItemManagement controller class.
|
||||||
|
|
|
||||||
|
|
@ -23,8 +23,6 @@ use phpOMS\DataStorage\Database\DataMapperAbstract;
|
||||||
* @license OMS License 1.0
|
* @license OMS License 1.0
|
||||||
* @link https://orange-management.org
|
* @link https://orange-management.org
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*
|
|
||||||
* @todo Do I really want to create a relation to the language mapper? It's not really needed right?
|
|
||||||
*/
|
*/
|
||||||
final class ItemAttributeMapper extends DataMapperAbstract
|
final class ItemAttributeMapper extends DataMapperAbstract
|
||||||
{
|
{
|
||||||
|
|
|
||||||
|
|
@ -24,8 +24,6 @@ use phpOMS\Localization\Defaults\LanguageMapper;
|
||||||
* @license OMS License 1.0
|
* @license OMS License 1.0
|
||||||
* @link https://orange-management.org
|
* @link https://orange-management.org
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*
|
|
||||||
* @todo Do I really want to create a relation to the language mapper? It's not really needed right?
|
|
||||||
*/
|
*/
|
||||||
final class ItemAttributeTypeL11nMapper extends DataMapperAbstract
|
final class ItemAttributeTypeL11nMapper extends DataMapperAbstract
|
||||||
{
|
{
|
||||||
|
|
@ -42,22 +40,6 @@ final class ItemAttributeTypeL11nMapper extends DataMapperAbstract
|
||||||
'itemmgmt_attr_type_l11n_lang' => ['name' => 'itemmgmt_attr_type_l11n_lang', 'type' => 'string', 'internal' => 'language'],
|
'itemmgmt_attr_type_l11n_lang' => ['name' => 'itemmgmt_attr_type_l11n_lang', 'type' => 'string', 'internal' => 'language'],
|
||||||
];
|
];
|
||||||
|
|
||||||
/**
|
|
||||||
* Has one relation.
|
|
||||||
*
|
|
||||||
* @var array<string, array{mapper:string, external:string, by?:string, column?:string, conditional?:bool}>
|
|
||||||
* @since 1.0.0
|
|
||||||
*/
|
|
||||||
protected static array $ownsOne = [
|
|
||||||
'language' => [
|
|
||||||
'mapper' => LanguageMapper::class,
|
|
||||||
'external' => 'itemmgmt_attr_type_l11n_lang',
|
|
||||||
'by' => 'code2',
|
|
||||||
'column' => 'code2',
|
|
||||||
'conditional' => true,
|
|
||||||
],
|
|
||||||
];
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Primary table.
|
* Primary table.
|
||||||
*
|
*
|
||||||
|
|
|
||||||
|
|
@ -25,8 +25,6 @@ use phpOMS\Localization\Defaults\LanguageMapper;
|
||||||
* @license OMS License 1.0
|
* @license OMS License 1.0
|
||||||
* @link https: //orange-management.org
|
* @link https: //orange-management.org
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*
|
|
||||||
* @todo Do I really want to create a relation to the language mapper? It's not really needed right?
|
|
||||||
*/
|
*/
|
||||||
final class ItemAttributeValueMapper extends DataMapperAbstract
|
final class ItemAttributeValueMapper extends DataMapperAbstract
|
||||||
{
|
{
|
||||||
|
|
@ -48,29 +46,6 @@ final class ItemAttributeValueMapper extends DataMapperAbstract
|
||||||
'itemmgmt_attr_value_country' => ['name' => 'itemmgmt_attr_value_country', 'type' => 'string', 'internal' => 'country'],
|
'itemmgmt_attr_value_country' => ['name' => 'itemmgmt_attr_value_country', 'type' => 'string', 'internal' => 'country'],
|
||||||
];
|
];
|
||||||
|
|
||||||
/**
|
|
||||||
* Has one relation.
|
|
||||||
*
|
|
||||||
* @var array<string, array{mapper:string, external:string, by?:string, column?:string, conditional?:bool}>
|
|
||||||
* @since 1.0.0
|
|
||||||
*/
|
|
||||||
protected static array $ownsOne = [
|
|
||||||
'language' => [
|
|
||||||
'mapper' => LanguageMapper::class,
|
|
||||||
'external' => 'itemmgmt_attr_value_lang',
|
|
||||||
'by' => 'code2',
|
|
||||||
'column' => 'code2',
|
|
||||||
'conditional' => true,
|
|
||||||
],
|
|
||||||
'country' => [
|
|
||||||
'mapper' => CountryMapper::class,
|
|
||||||
'external' => 'itemmgmt_attr_value_country',
|
|
||||||
'by' => 'code2',
|
|
||||||
'column' => 'code2',
|
|
||||||
'conditional' => true,
|
|
||||||
],
|
|
||||||
];
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Primary table.
|
* Primary table.
|
||||||
*
|
*
|
||||||
|
|
|
||||||
|
|
@ -24,8 +24,6 @@ use phpOMS\Localization\Defaults\LanguageMapper;
|
||||||
* @license OMS License 1.0
|
* @license OMS License 1.0
|
||||||
* @link https://orange-management.org
|
* @link https://orange-management.org
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*
|
|
||||||
* @todo Do I really want to create a relation to the language mapper? It's not really needed right?
|
|
||||||
*/
|
*/
|
||||||
final class ItemL11nMapper extends DataMapperAbstract
|
final class ItemL11nMapper extends DataMapperAbstract
|
||||||
{
|
{
|
||||||
|
|
@ -50,13 +48,6 @@ final class ItemL11nMapper extends DataMapperAbstract
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*/
|
*/
|
||||||
protected static array $ownsOne = [
|
protected static array $ownsOne = [
|
||||||
'language' => [
|
|
||||||
'mapper' => LanguageMapper::class,
|
|
||||||
'external' => 'itemmgmt_item_l11n_lang',
|
|
||||||
'by' => 'code2',
|
|
||||||
'column' => 'code2',
|
|
||||||
'conditional' => true,
|
|
||||||
],
|
|
||||||
'type' => [
|
'type' => [
|
||||||
'mapper' => ItemL11nTypeMapper::class,
|
'mapper' => ItemL11nTypeMapper::class,
|
||||||
'external' => 'itemmgmt_item_l11n_typeref',
|
'external' => 'itemmgmt_item_l11n_typeref',
|
||||||
|
|
|
||||||
|
|
@ -23,8 +23,6 @@ use phpOMS\DataStorage\Database\DataMapperAbstract;
|
||||||
* @license OMS License 1.0
|
* @license OMS License 1.0
|
||||||
* @link https://orange-management.org
|
* @link https://orange-management.org
|
||||||
* @since 1.0.0
|
* @since 1.0.0
|
||||||
*
|
|
||||||
* @todo Do I really want to create a relation to the language mapper? It's not really needed right?
|
|
||||||
*/
|
*/
|
||||||
final class ItemL11nTypeMapper extends DataMapperAbstract
|
final class ItemL11nTypeMapper extends DataMapperAbstract
|
||||||
{
|
{
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user