[Vtigercrm-developers] Vtiger 7.3 - Transfer ownership not working

[Uma S]

Agree! But this is a legacy code. Let's review to see if we can get it
centralized to vtiger_crmenityrel.

Please do leave your comment on the issue, which will be helpful in
followup.

On Wed, Nov 18, 2020 at 1:37 PM Martin Allen <martin.allen at clystnet.com>
wrote:

> +1 to that Alan. Relationships should ALL work in the same way. As the
> table name implies it should store the n:n relationship between entity
> modules (and that means ALL entity modules!)
>
> Martin Allen
>
>
> *01392 248692 - Main Office01392 690659 - Direct Line*
>
> *[image: signature2]*
>
> Have you visited our website recently? https://www.clystnet.com
> <http://www.clystnet.com/>
>
>
> The information in this email is confidential If you are not the intended
> recipient, you must not read or use that information. This email and any
> attachments are believed to be virus free however no responsibility is
> accepted by Clystnet for any loss or damage arising in any way from receipt
> or use thereof. Clystnet Ltd (company reg number 7164503) is based at
> Silverdown Park, Fair Oak Close, Clyst Honiton, EX5 2UX
>
>
> On Wed, 18 Nov 2020 at 07:57, Alan Lord <alanslists at gmail.com> wrote:
>
>> Why do we store contacts<->potentials in that table?
>>
>> Why not use crmentityrel? Then vtiger only needs one mechanism and no
>> "extra code" to handle these weird "non-standard" relationships?
>>
>> Al
>>
>>
>> On 18/11/2020 06:36, Uma S wrote:
>> > Hi Henry,
>> >
>> > Yes! We validated this issue and noted the cause as, Relationship
>> > between Contacts to Opportunities is not pushed
>> > to vtiger_contpotentialrel. Whereas transfer ownership solely depends
>> on
>> > these entries.
>> >
>> > I have created the issue here
>> > <https://code.vtiger.com/vtiger/vtigercrm/issues/1542>, Which will be
>> > addressed in the pipeline.
>> >
>> > On Wed, Nov 18, 2020 at 2:52 AM Alan Lord <alanslists at gmail.com
>> > <mailto:alanslists at gmail.com>> wrote:
>> >
>> >     On 17/11/2020 20:57, Henry Cumbicus Rivera wrote:
>> >      > Of course, Alan,
>> >      > the problem is that the getRelatedModuleRecordIds method only
>> >     searches
>> >      > the vtiger_contpotentialrel table
>> >
>> >     I'm not sure that is correct.
>> >
>> >     The use of vtiger_contpotentialrel will be called if the
>> >     $relationFieldId is empty.
>> >
>> >
>> https://code.vtiger.com/vtiger/vtigercrm/blob/master/modules/Vtiger/models/Module.php#L1653
>> >
>> >     But this test is itself inside a foreach loop so I think it should
>> >     check
>> >     _both_ relationship mechanisms anyway.
>> >
>> >
>> https://code.vtiger.com/vtiger/vtigercrm/blob/master/modules/Vtiger/models/Module.php#L1645
>> >
>> >     Al
>> >     _______________________________________________
>> >     http://www.vtiger.com/
>> >
>> >
>> >
>> > --
>> > With
>> > Best Regards
>> > Uma.S
>> > Vtiger Team
>> >
>> > _______________________________________________
>> > http://www.vtiger.com/
>> >
>> _______________________________________________
>> http://www.vtiger.com/
>
> _______________________________________________
> http://www.vtiger.com/



-- 
With
Best Regards
Uma.S
Vtiger Team
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20201118/41788485/attachment.html>