ZSearchInterface: DTML renders correctly, ZPT doesn't

Bug #142743 reported by Bug Importer
0
Affects Status Importance Assigned to Milestone
Zope 2
Invalid
Low
Unassigned

Bug Description

create a ZSI with the ZMI.
a) creating as dtml:
searching with the provided interface renders the correct results, even if using more then one page
b) creating as zpt:
searching with the provided interface -- renders the correct results if -- and only if -- the results fit on one page! if there are more results than fit on one page, the second, at the latest the third page shows complete mischief which has nothing in common with the search criteria.

Tags: bugday catalog
Revision history for this message
Andy Dustman (andy-dustman) wrote :

I don't know what this means, exactly, but when generating page templates, you get nested html elements, i.e.

<html><body>
<html>
  <body tal:define=...>
...

  </body>
</html>

</body></html>

Additionally, if you select multiple searchable objects when creating, you get one of these structures for each searchable object.

Revision history for this message
Andy Dustman (andy-dustman) wrote :

Here's the fix to Shared/DC/ZRDB/Search.py. It includes a small XHTML fix (<hr />), though I can't imagine what the point of doing multiple listing in one file is, and even then, you get <html>...</html><hr /><html>...</html>, which is problematic, but is better than all of that within <html><body>...</body></html>.

> --- Search.py.orig 2003-12-11 14:48:18.000000000 -0500
> +++ Search.py 2003-12-11 15:54:50.562614641 -0500
> @@ -98,11 +98,9 @@
>
> self.manage_addProduct['PageTemplates'].manage_addPageTemplate(
> report_id,report_title,
> - ('<html><body>\n%s\n'
> - '</body></html>' %
> join(map(lambda q, report_style=report_style:
> custom_default_zpt_report(q.id, q, no_table=report_style), qs),
> - '\n<hr>\n')))
> + '\n<hr />\n'))
>
> if REQUEST: return self.manage_main(self,REQUEST)
>

Revision history for this message
Andreas Jung (ajung) wrote :

Changes: edited transcript

Revision history for this message
Andreas Jung (ajung) wrote :

Changes: importance (critical => medium)

Tres Seaver (tseaver)
Changed in zope2:
status: New → Triaged
importance: Medium → Low
tags: added: bugday
removed: bug
Revision history for this message
Colin Watson (cjwatson) wrote :

The zope2 project on Launchpad has been archived at the request of the Zope developers (see https://answers.launchpad.net/launchpad/+question/683589 and https://answers.launchpad.net/launchpad/+question/685285). If this bug is still relevant, please refile it at https://github.com/zopefoundation/zope2.

Changed in zope2:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.