Re: [Bug?] Filtering on property=-1 and ordering by the same property at the same time

Nagy Gabor <[email protected]>
Newsgroups gmane.comp.bug-tracking.roundup.user
Message-ID <20210518231447.44490102@Dell>
> > 
> > In any
> > case I confirm that searching for -1 in link type fields
> > (assignedto) with sorting on the same field pretends that -1 is not
> > specified. Sorting by assignedto with filter:
> > 
> >     assignedto=-1,1,Admin1
> > 
> > behaves as though the filter was:
> > 
> >     assignedto=1,Admin1  
> 
> Should be fixed now.
> 
> > it's only the search using a negative pseudovalue along with some
> > other value that is problematic. Filter:
> > 
> >    assignedto=-1,-2
> > 
> > returns no results but I know there are issues without anybody
> > assigned.
> > 
> >    assignedto=1,-2  
> 
> Although Link expressions *never* worked I've chosen to implement
> them. I think it's hard to explain to users why it works for
> Multilink but not Link properties. If you're interested see how I've
> fixed negation: An expression like "not (x = 1)" will find all cases
> where x has a value other than 1 but will *not* find cases where x is
> NULL in SQL :-)
> 
> On Wed, May 12, 2021 at 07:37:31PM +0200, Nagy Gabor wrote:
> 
> > > > Symptom:
> > > > 
> > > > http://localhost:8080/tracker/issue?@filter=status&status=-1,2
> > > > 
> > > > lists all issues with status -1 (unset) or 2 (deferred).
> > > > 
> > > > However, in the list of
> > > > 
> > > > http://localhost:8080/tracker/issue?@filter=status&status=-1,2&@sort=status
> > > > 
> > > > issues with status -1 are missing.    
> 
> This should be fixed, let us know if it now works for you.
> And thanks for the good example that allowed me to quickly come up
> with a test for this.
> 
> Ralf

Dear Ralf,

Thank you for the quick fix.

However, it seems that there is a misunderstanding here. I am using
Roundup 2.0.0 (with some custom patches applies, which should not
affect this issue)... And this bug exists there, too, not only in the
devel branch. If you can provide me a patch for 2.0.0, I can test it.
(Btw, can the https://sourceforge.net/p/roundup/code/commit_browser
page generate .patch file for a particular commit? I cannot find that.)

Regards,
Gábor


_______________________________________________
Roundup-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/roundup-users
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.