Issue 74125

Summary:

ODF and OOo XML file format: Alien attributes get lost in text style properties

Product:

Writer

Reporter:

jogi

Component:

save-export

Assignee:

jogi

Status:

CLOSED FIXED

QA Contact:

issues@sw <issues>

Severity:

Trivial

Priority:

CC:

issues

Version:

680m198

Keywords:

oooqa, regression

Target Milestone:

---

Hardware:

All

OS:

All

Issue Type:

DEFECT

Latest Confirmation in:

---

Developer Difficulty:

---

Issue Depends on:

Issue Blocks:

73858

Attachments:

Description	Flags
test document	none
exported from OOo 2.1 - PASSED	none
exported from OOF680m5 - FAILED	none
correct content.xml (OOo 2.1)	none
wrong content.xml with missing alian attributes in text style attributes	none
wrong ODF content.xml	none

Description jogi 2007-02-02 13:20:08 UTC

testcase:
qa/qatesttool/xml/level1/writer_xml_7_export.bas -> tfiscus

Found in version(s):
680m201 (Build:9112)	Hidden attributes (in a text style): attr1 is not 42!
680m201 (Build:9112)	Hidden attributes (in a text style): bar:attr3 is not 44!
680m201 (Build:9112)	Hidden attributes (in a text style): foo:attr2 is not 43!
680m198 (Build:9104)	Hidden attributes (in a text style): attr1 is not 42!
680m198 (Build:9104)	Hidden attributes (in a text style): bar:attr3 is not 44!
680m198 (Build:9104)	Hidden attributes (in a text style): foo:attr2 is not 43!

manual short description of test case:

- Open test document
- Saving (and if already exiting overwriting) the document.
- Closing the document also if there is a verification dialog.
- Look in content.xml (with a XML editor is the best way)
Hidden Namespaces Searching for: xmlns:foo="http://www.foo.com" Searching for:
xmlns:bar="http://www.bar.com"
hidden attributes in a table style Searching for: attr1="42" Searching for:
foo:attr2="43" Searching for: bar:attr3="44"
hidden attributes in a table cell style Searching for: attr1="42" Searching for:
foo:attr2="43" Searching for: bar:attr3="44"
hidden attributes in a paragraph style Searching for: attr1="42" Searching for:
foo:attr2="43" Searching for: bar:attr3="44"
hidden attributes in a text style Searching for: attr1="42" Searching for:
foo:attr2="43" Searching for: bar:attr3="44"
hidden attributes in a frame style Searching for: attr1="42" Searching for:
foo:attr2="43" Searching for: bar:attr3="44"
- in STYLES.XML
Hidden Namespace Searching for: xmlns:foo="http://www.foo.com" Searching for:
xmlns:bar="http://www.bar.com"
hidden attributes in the template style Â´Text bodyÂ´ Searching for: attr1="42"
Searching for: foo:attr2="43" Searching for: bar:attr3="44"

Comment 1 jogi 2007-02-02 13:23:33 UTC

Created attachment 42669 [details]
test document

Comment 2 jogi 2007-02-02 13:25:34 UTC

Created attachment 42670 [details]
exported from OOo 2.1 - PASSED

Comment 3 jogi 2007-02-02 13:26:44 UTC

Created attachment 42671 [details]
exported from OOF680m5 - FAILED

Comment 4 jogi 2007-02-02 13:28:32 UTC

Created attachment 42672 [details]
correct content.xml (OOo 2.1)

Comment 5 jogi 2007-02-02 13:29:42 UTC

Created attachment 42673 [details]
wrong content.xml with missing alian attributes in text style attributes

Comment 6 frank.meies 2007-02-02 15:58:01 UTC

FME: The alien attributes are located in the automatic character styles.
Automatic styles (in Writer core, not in the file format) are currently not able
to contain alien attributes. This broke with cws swautomatic01. Btw: User
defined character styles cannot contain alien attributes either, but this did
not break with swautomatic01, this never seemed to work correctly.

Comment 7 jogi 2007-02-05 06:41:02 UTC

AFAIK and AFAIR: Alien attributes were supported in automatic-styles only and
never in user defined styles. There has never been written a testcase for
(neither development and testing), so if there was a support I would wondering.

Comment 8 jogi 2007-02-05 10:16:59 UTC

Also ODF affected.

Comment 9 jogi 2007-02-05 10:18:34 UTC

Created attachment 42745 [details]
wrong ODF content.xml

Comment 10 frank.meies 2007-02-05 10:28:22 UTC

FME: Fixed in cws swqbf96:

/cvs/sw/sw/inc/hintids.hxx,v  <--  hintids.hxx

new revision: 1.31.96.1; previous revision: 1.31

/cvs/sw/sw/source/core/bastyp/init.cxx,v  <--  init.cxx

new revision: 1.58.68.1; previous revision: 1.58

/cvs/sw/sw/source/core/txtnode/thints.cxx,v  <--  thints.cxx

new revision: 1.43.36.1.14.1; previous revision: 1.43.36.1

/cvs/sw/sw/source/core/unocore/unomap.cxx,v  <--  unomap.cxx

new revision: 1.192.36.1.14.1; previous revision: 1.192.36.1

/cvs/sw/sw/source/core/unocore/unostyle.cxx,v  <--  unostyle.cxx

new revision: 1.68.94.1; previous revision: 1.68

Comment 11 frank.meies 2007-02-05 12:30:06 UTC

Ready for QA.

Comment 12 jogi 2007-02-08 06:56:50 UTC

verified because XML test passed

Comment 13 ace_dent 2008-05-16 03:34:30 UTC

This Issue is 'Verified' and not updated in 1yr+, so Closing.
A Closed Issue is a Happy Issue (TM).

Regards,
Andrew
 
Cleaning-up and Closing old Issues as part of:
~ The Grand Bug Squash, pre v3 ~