Skip to content

Commit 109676e

Browse files
ENGCOM-8722: fix for ambigous column user_id in where clause #31205
2 parents 999fb79 + 764448d commit 109676e

File tree

2 files changed

+31
-0
lines changed

2 files changed

+31
-0
lines changed

app/code/Magento/User/Model/ResourceModel/User/Collection.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,5 +41,6 @@ protected function _initSelect()
4141
'user_role.parent_id = detail_role.role_id',
4242
['role_name']
4343
);
44+
$this->addFilterToMap('user_id', 'main_table.user_id');
4445
}
4546
}
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
namespace Magento\User\Model\ResourceModel\User;
7+
8+
/**
9+
* User collection test
10+
* @magentoAppArea adminhtml
11+
*/
12+
class CollectionTest extends \PHPUnit\Framework\TestCase
13+
{
14+
/**
15+
* @var \Magento\User\Model\ResourceModel\User\Collection
16+
*/
17+
protected $_collection;
18+
19+
protected function setUp(): void
20+
{
21+
$this->_collection = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
22+
\Magento\User\Model\ResourceModel\User\Collection::class
23+
);
24+
}
25+
26+
public function testFilteringCollectionByUserId()
27+
{
28+
$this->assertEquals(1, $this->_collection->addFieldToFilter('user_id', 1)->count());
29+
}
30+
}

0 commit comments

Comments
 (0)