Bug 478280

Summary: complex filters get unecessarily complex when converted in FilterBuilder
Product: [Retired] penrose Reporter: Chandrasekar Kannan <ckannan>
Component: EngineAssignee: Endi Sukma Dewata <edewata>
Status: CLOSED EOL QA Contact: Ben Levenson <benl>
Severity: medium Docs Contact:
Priority: medium    
Version: 2.0CC: benl, nmalki, ykaul
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-03-27 19:41:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 471500    

Description Chandrasekar Kannan 2008-12-27 08:14:05 UTC
In the master branch in git at the moment a complex filter get's incorrectly converted. For example the filter '(&(uid=kouk)(givenName=Konstantinos))' will become '(&(uid=kouk)(&(uid=kouk)(givenName=Konstantinos)))'. In this case there is no error because the filters are logically equivalent. But this might not always be the case. The attached patch fixes this behaviour.

=========================================================
Issue dump from jira
$VAR1 = {
          'priority' => '4',
          'customFieldValues' => [],
          'project' => 'PENROSE',
          'status' => '1',
          'components' => [
                            {
                              'name' => 'Engine',
                              'id' => '10009'
                            }
                          ],
          'attachmentNames' => '0002-recalculate-variables-when-converting-complex-filter.patch',
          'reporter' => 'kouk',
          'key' => 'PENROSE-315',
          'assignee' => 'jimyang',
          'summary' => 'complex filters get unecessarily complex when converted in FilterBuilder',
          'id' => '11042',
          'updated' => '2008-08-25 10:43:43.0',
          'votes' => '0',
          'fixVersions' => [],
          'affectsVersions' => [],
          'description' => 'In the master branch in git at the moment a complex filter get's incorrectly converted. For example the filter '(&(uid=kouk)(givenName=Konstantinos))' will become '(&(uid=kouk)(&(uid=kouk)(givenName=Konstantinos)))'. In this case there is no error because the filters are logically equivalent. But this might not always be the case. The attached patch fixes this behaviour.',
          'created' => '2008-08-25 10:43:17.0',
          'type' => '1'
        };


=========================================================

Comment 1 Endi Sukma Dewata 2009-02-05 18:08:18 UTC
Might be related to https://bugzilla.redhat.com/show_bug.cgi?id=478255.

Comment 2 Endi Sukma Dewata 2009-02-24 00:30:15 UTC
We need the configuration files and the query that was sent to the server.