Issue 77913 - settings.xml parser should be wrap spaces
Summary: settings.xml parser should be wrap spaces
Status: ACCEPTED
Alias: None
Product: xml
Classification: Code
Component: code (show other issues)
Version: OOo 2.2
Hardware: All All
: P4 Trivial (vote)
Target Milestone: 4.x
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-30 07:43 UTC by ttissino
Modified: 2013-02-07 21:56 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
sample file (5.99 KB, application/vnd.oasis.opendocument.text)
2007-05-30 07:45 UTC, ttissino 		no flags Details
the same file, but settings.xml with breaklines and spaces. (6.01 KB, application/vnd.oasis.opendocument.text)
2007-05-30 07:47 UTC, ttissino 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description ttissino 2007-05-30 07:43:17 UTC 
I noticed that OOo doesn't handle correctly settings.xml, if that file contains
spaces. For example, each of these two elements should set the value "true" to
variable "AddParaTableSpacing":

<config:config-item config:name="AddParaTableSpacing"
config:type="boolean">true</config:config-item>

   <config:config-item config:name="AddParaTableSpacing" config:type="boolean">
    true
   </config:config-item>

Instead, the second form doesn't work.
Comment 1 ttissino 2007-05-30 07:45:57 UTC 
Created attachment 45508 [details]
sample file
Comment 2 ttissino 2007-05-30 07:47:52 UTC 
Created attachment 45509 [details]
the same file, but settings.xml with breaklines and spaces.
Comment 3 jogi 2007-06-19 06:08:15 UTC 
@mib: I haven't seen this.
Comment 4 michael.brauer 2007-07-06 12:44:12 UTC 
The ODF spec does not say whether spaces are allowed or not, so it is probably
safe to assume they are not allowed. However, it is of cause reasonable to
remove them, as this is what would happen if the value would be an XSD boolean
datatype.
Comment 5 michael.brauer 2007-07-06 13:01:11 UTC 
changed target
Comment 6 michael.brauer 2007-12-03 10:59:25 UTC 
new target
Comment 7 michael.brauer 2008-06-04 07:20:27 UTC 
Adapted priority and target.