Adding an address directly to the account

I am trying to create a link between addresses and acounts on or CRM 2.
So I added 2 fields with a migration, “shipping address” and “billing address”.

PHP

$table->addColumn( 'shipping_address_id', 'integer', [ 'notnull' => false, 'oro_options' => [ 'extend' => [ 'is_extended' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'datagrid' => ['is_visible' => true], 'entity' => ['label' => 'Shipping address'], ] ] ); $table->addColumn( 'billing_address_id', 'integer', [ 'notnull' => false, 'oro_options' => [ 'extend' => [ 'is_extended' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'datagrid' => ['is_visible' => true], 'entity' => ['label' => 'Billing address'], ] ] ); $table->addIndex(['shipping_address_id'], 'IDX_7166D3714D4CFF2B', []); $table->addIndex(['billing_address_id'], 'IDX_7166D37179D0C0E4', []); $table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['shipping_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] ); $table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['billing_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] );

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46

$table->addColumn(             'shipping_address_id',             'integer',             [                 'notnull' => false,                 'oro_options' => [                     'extend' => [                         'is_extended' => true,                         'owner' => ExtendScope::OWNER_CUSTOM,                     ],                     'datagrid' => ['is_visible' => true],                     'entity' => ['label' => 'Shipping address'],                 ]             ]         );         $table->addColumn(             'billing_address_id',             'integer',             [                 'notnull' => false,                 'oro_options' => [                     'extend' => [                         'is_extended' => true,                         'owner' => ExtendScope::OWNER_CUSTOM,                     ],                     'datagrid' => ['is_visible' => true],                     'entity' => ['label' => 'Billing address'],                 ]             ]         );   $table->addIndex(['shipping_address_id'], 'IDX_7166D3714D4CFF2B', []);                 $table->addIndex(['billing_address_id'], 'IDX_7166D37179D0C0E4', []);                   $table->addForeignKeyConstraint(                     $schema->getTable('oro_address'),                     ['shipping_address_id'],                     ['id'],                     ['onDelete' => 'SET NULL', 'onUpdate' => null]                 );                 $table->addForeignKeyConstraint(                     $schema->getTable('oro_address'),                     ['billing_address_id'],                     ['id'],                     ['onDelete' => 'SET NULL', 'onUpdate' => null]                 );

I also added the following code in account/update.html.twig

PHP

{% set dataBlocks = dataBlocks|merge( [{ 'title' : 'Addresses'|trans, 'subblocks': [ { 'title': 'oro.account.billing_address_id.label'|trans, 'data': [form_widget(form.billing_address_id)] }, { 'title': 'oro.account.shipping_address_id.label'|trans, 'data': [form_widget(form.shipping_address_id)] }, ] }] ) %}

1 2 3 4 5 6 7 8 9 10 11 12 13

{% set dataBlocks = dataBlocks|merge( [{         'title' : 'Addresses'|trans,         'subblocks': [         {             'title': 'oro.account.billing_address_id.label'|trans,             'data': [form_widget(form.billing_address_id)]         },         {             'title': 'oro.account.shipping_address_id.label'|trans,             'data': [form_widget(form.shipping_address_id)]         },         ]     }] ) %}

I also added the following code in form/type/accountType.php

PHP

$builder->add( 'shipping_address_id',//shippingAddress 'oro_address', [ 'required' => false ] ); $builder->add( 'billing_address_id',//billingAddress 'oro_address', [ 'required' => false ] ); $this->propertyAccessor = new PropertyAccessor(); $builder->addModelTransformer(new AddressSameTransformer( $this->propertyAccessor, ['billing_address_id', 'shipping_address_id'] ));

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

$builder->add(             'shipping_address_id',//shippingAddress             'oro_address',             [                 'required' => false             ]         );         $builder->add(             'billing_address_id',//billingAddress             'oro_address',             [                 'required' => false             ]         );           $this->propertyAccessor = new PropertyAccessor();         $builder->addModelTransformer(new AddressSameTransformer(             $this->propertyAccessor,             ['billing_address_id', 'shipping_address_id']         ));

I can now see an “address” block in the edit view, and I twice get a form to enter the address in: perfect!
The only problem is, that when I save I get the error I get the message “This value should contain only numbers.”
I assume I did something wrong with my migration and should have used a different type than “integer”, but I do not know what.
Can somebody please help me?

Topic

[Nikita MakarovParticipant]

Greetings keelen.

I assume you forgot to add foreign-keys for your fields, for example

PHP

$table->addForeignKeyConstraint( $schema->getTable('TABLE_NAME_WHERE_ADDERESES_STORED'), ['shipping_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] ); $table->addForeignKeyConstraint( $schema->getTable('TABLE_NAME_WHERE_ADDERESES_STORED'), ['billing_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] );

1 2 3 4 5 6 7 8 9 10 11 12

$table->addForeignKeyConstraint(             $schema->getTable('TABLE_NAME_WHERE_ADDERESES_STORED'),             ['shipping_address_id'],             ['id'],             ['onDelete' => 'SET NULL', 'onUpdate' => null]         );         $table->addForeignKeyConstraint(             $schema->getTable('TABLE_NAME_WHERE_ADDERESES_STORED'),             ['billing_address_id'],             ['id'],             ['onDelete' => 'SET NULL', 'onUpdate' => null]         );

As example can be used following code

[keelenParticipant]

Hey, thank you for your response.

I did add those foreign keys in my migration like you can see in my code snippet above. I have:

PHP

$table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['shipping_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] ); $table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['billing_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] );

1 2 3 4 5 6 7 8 9 10 11 12

$table->addForeignKeyConstraint(                     $schema->getTable('oro_address'),                     ['shipping_address_id'],                     ['id'],                     ['onDelete' => 'SET NULL', 'onUpdate' => null]                 );                 $table->addForeignKeyConstraint(                     $schema->getTable('oro_address'),                     ['billing_address_id'],                     ['id'],                     ['onDelete' => 'SET NULL', 'onUpdate' => null]                 );

[Nikita MakarovParticipant]

Could you please provide a full migration source code?
Also please let me know how you apply migrations?
Or you reinstall application each time?

And please specify the version of CRM application you are using

[keelenParticipant]

This is my version installed:

oro/platform 2.4.4
oro/crm 2.4.4 (enterprise version)

I run migrations with the command:
php app/console oro:migration:load –force –timeout=0

I do NOT reinstall the application each time, just run the migration.
I also see that the migration succeeded in creating the 2 new fields in the account table – no problem there.

My full migration script:

PHP

<?php namespace LCL\AppBundle\Migrations\Schema\v1_7; use Doctrine\DBAL\Schema\Schema; use Oro\Bundle\EntityExtendBundle\EntityConfig\ExtendScope; use Oro\Bundle\EntityExtendBundle\Migration\Extension\ExtendExtension; use Oro\Bundle\EntityExtendBundle\Migration\Extension\ExtendExtensionAwareInterface; use Oro\Bundle\MigrationBundle\Migration\Migration; use Oro\Bundle\MigrationBundle\Migration\QueryBag; class AddAddressToAccount implements Migration, ExtendExtensionAwareInterface { /** @var ExtendExtension */ private $extendExtension; public function setExtendExtension(ExtendExtension $extendExtension) { $this->extendExtension = $extendExtension; } public function up(Schema $schema, QueryBag $queries) { $table = $schema->getTable('orocrm_account'); $table->addColumn( 'shipping_address_id', 'integer', [ 'notnull' => false, 'oro_options' => [ 'extend' => [ 'is_extended' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'datagrid' => ['is_visible' => true], 'entity' => ['label' => 'Shipping address'], ] ] ); $table->addColumn( 'billing_address_id', 'integer', [ 'notnull' => false, 'oro_options' => [ 'extend' => [ 'is_extended' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'datagrid' => ['is_visible' => true], 'entity' => ['label' => 'Billing address'], ] ] ); $table->addIndex(['shipping_address_id'], 'IDX_7166D3714D4CFF2B', []); $table->addIndex(['billing_address_id'], 'IDX_7166D37179D0C0E4', []); $table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['shipping_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] ); $table->addForeignKeyConstraint( $schema->getTable('oro_address'), ['billing_address_id'], ['id'], ['onDelete' => 'SET NULL', 'onUpdate' => null] ); } }

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76

<?php   namespace LCL\AppBundle\Migrations\Schema\v1_7;   use Doctrine\DBAL\Schema\Schema; use Oro\Bundle\EntityExtendBundle\EntityConfig\ExtendScope; use Oro\Bundle\EntityExtendBundle\Migration\Extension\ExtendExtension; use Oro\Bundle\EntityExtendBundle\Migration\Extension\ExtendExtensionAwareInterface; use Oro\Bundle\MigrationBundle\Migration\Migration; use Oro\Bundle\MigrationBundle\Migration\QueryBag;   class AddAddressToAccount implements Migration, ExtendExtensionAwareInterface {     /** @var  ExtendExtension */     private $extendExtension;       public function setExtendExtension(ExtendExtension $extendExtension)     {         $this->extendExtension = $extendExtension;     }       public function up(Schema $schema, QueryBag $queries)     {         $table = $schema->getTable('orocrm_account');           $table->addColumn(             'shipping_address_id',             'integer',             [                 'notnull' => false,                 'oro_options' => [                     'extend' => [                         'is_extended' => true,                         'owner' => ExtendScope::OWNER_CUSTOM,                     ],                     'datagrid' => ['is_visible' => true],                     'entity' => ['label' => 'Shipping address'],                 ]             ]         );           $table->addColumn(             'billing_address_id',             'integer',             [                 'notnull' => false,                 'oro_options' => [                     'extend' => [                         'is_extended' => true,                         'owner' => ExtendScope::OWNER_CUSTOM,                     ],                     'datagrid' => ['is_visible' => true],                     'entity' => ['label' => 'Billing address'],                 ]             ]         );             $table->addIndex(['shipping_address_id'], 'IDX_7166D3714D4CFF2B', []);         $table->addIndex(['billing_address_id'], 'IDX_7166D37179D0C0E4', []);           $table->addForeignKeyConstraint(             $schema->getTable('oro_address'),             ['shipping_address_id'],             ['id'],             ['onDelete' => 'SET NULL', 'onUpdate' => null]         );         $table->addForeignKeyConstraint(             $schema->getTable('oro_address'),             ['billing_address_id'],             ['id'],             ['onDelete' => 'SET NULL', 'onUpdate' => null]         );       } }

[keelenParticipant]

Do I maybe need to add something to a the Account entity like:

PHP

/** * @var Address $shippingAddress * * @ORM\ManyToOne(targetEntity="Oro\Bundle\AddressBundle\Entity\Address", cascade={"persist", "remove"}) * @ORM\JoinColumn(name="shipping_address_id", referencedColumnName="id", onDelete="SET NULL") * @ConfigField(defaultValues={"merge"={"enable"=true}}) */ protected $shippingAddress; /** * @var Address $billingAddress * * @ORM\ManyToOne(targetEntity="Oro\Bundle\AddressBundle\Entity\Address", cascade={"persist", "remove"}) * @ORM\JoinColumn(name="billing_address_id", referencedColumnName="id", onDelete="SET NULL") * @ConfigField(defaultValues={"merge"={"enable"=true}}) * @ConfigField(defaultValues={"merge"={"enable"=true}}) */ protected $billingAddress;

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

/**      * @var Address $shippingAddress      *      * @ORM\ManyToOne(targetEntity="Oro\Bundle\AddressBundle\Entity\Address", cascade={"persist", "remove"})      * @ORM\JoinColumn(name="shipping_address_id", referencedColumnName="id", onDelete="SET NULL")      * @ConfigField(defaultValues={"merge"={"enable"=true}})      */     protected $shippingAddress;     /**      * @var Address $billingAddress      *      * @ORM\ManyToOne(targetEntity="Oro\Bundle\AddressBundle\Entity\Address", cascade={"persist", "remove"})      * @ORM\JoinColumn(name="billing_address_id", referencedColumnName="id", onDelete="SET NULL")      * @ConfigField(defaultValues={"merge"={"enable"=true}})      * @ConfigField(defaultValues={"merge"={"enable"=true}})      */     protected $billingAddress;

I see that in an old version of Oro CRM (februari 2014) there used to be a relation between address and account, but a later version removed that link. See commit https://github.com/oroinc/crm/blob/2d9e3a52cf7f1cbf22d400004ea776ce44777429/src/OroCRM/Bundle/AccountBundle/Entity/Account.php

In that commit there is a relation inside the docrine entity, like i linked above. Should I explore this direction, or should I only add relations with Migrations?

[Nikita MakarovParticipant]

I assume that maybe should be used proper field assignment

PHP

$this->extendExtension->addManyToOneRelation( $schema, 'orocrm_account', 'address', 'oro_address', 'label', [ 'extend' => [ 'is_extend' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'entity' => ['label' => 'Billing address'], 'datagrid' => [ 'is_visible' => DatagridScope::IS_VISIBLE_TRUE, ] ] );

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

$this->extendExtension->addManyToOneRelation(             $schema,             'orocrm_account',             'address',             'oro_address',             'label',             [                 'extend' => [                     'is_extend' => true,                     'owner' => ExtendScope::OWNER_CUSTOM,                 ],                 'entity' => ['label' => 'Billing address'],                 'datagrid' => [                     'is_visible' => DatagridScope::IS_VISIBLE_TRUE,                 ]             ]         );

but I’m checking it right now

[keelenParticipant]

Hey, I tried to ad two relations but that didn’t work either – it continued to give the error “This value should contain only numbers.”

PHP

// relation ship $this->extendExtension->addManyToOneRelation( $schema, 'orocrm_account', 'shippingAddress', 'oro_address', 'label', [ 'extend' => [ 'is_extend' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'entity' => ['label' => 'Shipping address'], 'datagrid' => [ 'is_visible' => DatagridScope::IS_VISIBLE_TRUE, ] ] ); $this->extendExtension->addManyToOneRelation( $schema, 'orocrm_account', 'billingAddress', 'oro_address', 'label', [ 'extend' => [ 'is_extend' => true, 'owner' => ExtendScope::OWNER_CUSTOM, ], 'entity' => ['label' => 'Billing address'], 'datagrid' => [ 'is_visible' => DatagridScope::IS_VISIBLE_TRUE, ] ] );

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36

// relation ship         $this->extendExtension->addManyToOneRelation(             $schema,             'orocrm_account',             'shippingAddress',             'oro_address',             'label',             [                 'extend' => [                     'is_extend' => true,                     'owner' => ExtendScope::OWNER_CUSTOM,                 ],                 'entity' => ['label' => 'Shipping address'],                 'datagrid' => [                     'is_visible' => DatagridScope::IS_VISIBLE_TRUE,                 ]             ]         );           $this->extendExtension->addManyToOneRelation(             $schema,             'orocrm_account',             'billingAddress',             'oro_address',             'label',             [                 'extend' => [                     'is_extend' => true,                     'owner' => ExtendScope::OWNER_CUSTOM,                 ],                 'entity' => ['label' => 'Billing address'],                 'datagrid' => [                     'is_visible' => DatagridScope::IS_VISIBLE_TRUE,                 ]             ]         );

Should I maybe open a ticket with the enterprise ticketing system?

[acalimanParticipant]

Hi keelen,

Yes, if you use the enterprise edition please open a new ticket on http://support.orocommerce.com for getting support.

Adrian

[Author]

Replies