Bug 1083341 - DocBook imports allow you to select a blank file
Summary: DocBook imports allow you to select a blank file
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: ImportTool
Version: 1.5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.5
Assignee: Matthew Casperson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-02 02:38 UTC by Lee Newson
Modified: 2014-08-04 22:28 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-05-01 21:40:01 UTC
Embargoed:

Attachments (Terms of Use)
Example book (2.70 MB, application/zip)
2014-04-02 06:30 UTC, Lee Newson 		no flags Details
View All

Description Lee Newson 2014-04-02 02:38:57 UTC 
If you try and use the DocBook 4.5 or 5.0 import mechanism, then when you get to the "Select the main XML file" part, the default selected value is a null value. If you just continue on like this the import starts fine and then just freezes. The following error shows up in the console:

qnazipmodel.js:46 Uncaught TypeError: Cannot call method 'toLowerCase' of null
Comment 1 Lee Newson 2014-04-02 02:56:30 UTC 
Actually this occurs with publican books as well. To replicate it for publican books I accidentally created a dir structure like so:

publican.cfg
es-ES
  es-ES
    mainfile.xml
    mainfile.ent
Comment 3 Matthew Casperson 2014-04-02 04:45:35 UTC 
An error message is now displayed: "Please select the main XML file before continuing"
Comment 4 Matthew Casperson 2014-04-02 04:45:56 UTC 
fixed in 201404021441
Comment 5 Lee Newson 2014-04-02 05:04:25 UTC 
Verified
Comment 6 Lee Newson 2014-04-02 06:30:23 UTC 
Created attachment 881641 [details]
Example book

Just had a case when trying to import the attached DocBook 5.0 book that it let me select a blank file without any error message.
Comment 7 Matthew Casperson 2014-04-02 21:14:17 UTC 
Fixed in 201404030704
Comment 8 Lee Newson 2014-04-02 22:36:11 UTC 
Verified
Note You need to log in before you can comment on or make changes to this bug.