[PLIP-Advisories] [Plone] #9352: Improved search results

Change notifications for Plone PLIPs on Trac. plone-plip-advisories at lists.plone.org
Mon Jun 20 23:39:40 UTC 2011


#9352: Improved search results
----------------------------+-----------------------------------------------
 Reporter:  laurenskling    |       Owner:  spliter 
     Type:  PLIP            |      Status:  assigned
 Priority:  major           |   Milestone:  4.2     
Component:  Infrastructure  |    Keywords:  search  
----------------------------+-----------------------------------------------

Comment(by rossp):

 Replying to [comment:56 davisagli]:
 > Replying to [comment:55 rossp]:
 > > (In [46338]) Add a review note about reusing plone.app.querystring.
 Refs #9352.
 > >
 > > It occurrs to me that this PLIP could have and probably should have
 > > re-used the plone.app.querystring package created for
 > > plone.app.collection which provides their AJAX search query
 > > building.
 >
 > IMHO a javascript-based query builder like that is appropriate for a
 site admin function like editing a collection, but not for an end-user
 function like buildout a search, which should still function with
 javascript disabled.

 I wasn't necessarily suggesting we expose the entire dynamic query
 building support of the package, but re-use the common bits.  There end up
 being a lot of edge case bugs with dynamic stuff like this, and it's a
 shame to fix the bugs in two places.  Especially given that p.a.search is
 still so buggy.

-- 
Ticket URL: <http://dev.plone.org/plone/ticket/9352#comment:116>
Plone <http://plone.org>
Plone Enterprise Content Management System


More information about the PLIP-Advisories mailing list