Ross Patterson wrote:
SimO <[email protected]> writes:

Ross Patterson wrote:
SimO <[email protected]> writes:

i.e: I set a range 0:100 and it shows me 5 items. If I put in the
portlet a second value of 80, instead of showing me 4 items it shows
NO items and worst, even if a re-insert 100 in the second value
nothing happens and I have to reload the page to get my items. The
weird thing is that if I set 0:80 trough the criteria tab it works
fine and it shows me 4 items.
Sounds like there's a bug in the integer range criterion.  I've added
this to the TODO:

  http://dev.plone.org/collective/changeset/82270>
I'll get to fixing it as soon as I can.

Thanks for the bug report!
Ross
Hi Ross,

thanks to you for this useful product. BTW I think there's a second bug  :)

When you look at the "Criteria type" list you see TWO items for setting:

Integer value or range
Integer value or range
List of values
List of values
Select values from list
Select values from list

I found that for every couple the first is gives you the base setting
without the "Form fields" section, thus not allowing you to show the
field in the search. The second is the right one.

This is because the ATCT non-form criteria exist along side the
collective.formcriteria criteria to preserve compatibility for
previously existing topics and criteria.

In the default installation of collective.formcriteria, however, the
JavaScript should narrow the list down to just the
collective.formcriteria version of the crtierion when you select the
field to use.  Can you please check if this works for you?  If it does,
then I don't think this is a bug.

If 'default installation' means to put 'collective.formcriteria' in the buildout - as I think :) - then, yes, I did so and I got this problem.

In order to avoid possible conflicts with some JS of mine or other sort of customizations I made, I just created a vanilla virtualenv (maybe too much... :) ) with a vanilla plone-3.1.7 instance where I installed ONLY collective.formcriteria. I added an IntegerField to ATDocument and it seems that such a problem (duplicated items) happens only with this kind of field/criteria.

One could argue that collective.formcriteria should use different text
for these descriptions but then when you looked at the JS narrowed list,
you'd see only options with the different text.  And since form criteria
don't *have* to appear on the form, appending "(form)" or some such
might be just as misleading.  If someone has a suggestion that can
resolve this, I'm all ears.  :)

The root of this problem is whether to outright replace the ATCT
criteria or preserve compatibility with ATCT topics and criteria.  I
opted for compatibility.

Ross
_______________________________________________
Product-Developers mailing list
[email protected]
http://lists.plone.org/mailman/listinfo/product-developers

Reply via email to