Bug 614728 - Publican does not warn about empty <term>
Publican does not warn about empty <term>
Status: CLOSED ERRATA
Product: Publican
Classification: Community
Component: publican (Show other bugs)
1.6
All Linux
low Severity medium
: ---
: ---
Assigned To: Jeff Fearn
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-15 01:52 EDT by Laura Bailey
Modified: 2010-11-23 23:16 EST (History)
4 users (show)

See Also:
Fixed In Version: publican-2.2-0.fc13
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-10-08 16:41:53 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
fop error on empty term (12.69 KB, text/plain)
2010-07-15 01:53 EDT, Laura Bailey
no flags Details

  None (edit)
Description Laura Bailey 2010-07-15 01:52:25 EDT
Description of problem:
When a <term/> tag is empty, or contains only other tags (<varname/>, etc.), Publican does not complain of an empty tag, but FOP falls over while building the PDF.

Version-Release number of selected component (if applicable):
publican-2.1-0.fc12.noarch

How reproducible:
Every time.

Steps to Reproduce:
1. Have a <term/> tag that is empty except for other tags:
<varlistentry>
  <term><varname></varname></term>
  <listitem> ...
or that is just empty:
 <varlistentry>
   <term></term>
   <listitem> ...

2. publican build --langs en-US --formats pdf
  
Actual results:
javax.xml.transform.TransformerException - see output in attached file.

Expected results:
Publican should make mention of the (technically valid) empty tag, or FOP should give a more informative error message.

Additional info:
Comment 1 Laura Bailey 2010-07-15 01:53:48 EDT
Created attachment 431971 [details]
fop error on empty term

Fop error for empty term tag attached.
Comment 2 Jeff Fearn 2010-07-27 01:52:04 EDT
Well it's clearly valid, if crazy, to have an empty term, so the PDF should build.

Fixed PDF build so it doesn't blow up on empty terms.

Fixed in build: 2.1-0%{?dist}.t2
Comment 3 Laura Bailey 2010-07-27 02:05:44 EDT
Thanks, Jeff. :)

Any chance we can get it to warn when it senses something like an empty term? I can't think of a circumstance in which an empty term would be correct, even if it's valid.
Comment 4 Jeff Fearn 2010-07-27 02:29:20 EDT
(In reply to comment #3)
> Thanks, Jeff. :)
> 
> Any chance we can get it to warn when it senses something like an empty term? I
> can't think of a circumstance in which an empty term would be correct, even if
> it's valid.    

I think it would be best to open a discussion upstream to get the DTD changed so that an empty term isn't valid.
Comment 5 Ruediger Landmann 2010-08-06 00:28:44 EDT
Verified in 2.1.t20

PDF builds fine even with an empty <term>
Comment 6 Fedora Update System 2010-10-06 01:48:14 EDT
publican-2.2-0.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/publican-2.2-0.fc13
Comment 7 Fedora Update System 2010-10-06 01:49:32 EDT
publican-2.2-0.fc12 has been submitted as an update for Fedora 12.
https://admin.fedoraproject.org/updates/publican-2.2-0.fc12
Comment 8 Fedora Update System 2010-10-06 01:49:35 EDT
publican-2.2-0.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/publican-2.2-0.fc14
Comment 9 Fedora Update System 2010-10-08 16:39:39 EDT
publican-2.2-0.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.