Issue 102701 - Some documents can not be saved any more successfuly.
Summary: Some documents can not be saved any more successfuly.
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m50
Hardware: All All
: P1 (highest) Trivial (vote)
Target Milestone: OOo 3.2
Assignee: oc
QA Contact: issues@framework
URL:
Keywords:
: 102755 103054 103110 (view as issue list)
Depends on:
Blocks:
 
Reported: 2009-06-12 09:29 UTC by oc
Modified: 2009-07-16 16:56 UTC (History)
4 users (show)

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


Attachments
Sampledoc with chart (19.51 KB, application/vnd.oasis.opendocument.spreadsheet)
2009-06-12 09:31 UTC, oc
no flags Details
The patch introducing missing changes. (2.96 KB, text/plain)
2009-06-12 15:38 UTC, mikhail.voytenko
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description oc 2009-06-12 09:29:08 UTC
In DEV300_m50 load a spreadsheet document with a chart in it (which was not
created by DEV300_m50) and try to save it to a different location => I/O Error
Comment 1 oc 2009-06-12 09:31:14 UTC
Created attachment 62941 [details]
Sampledoc with chart
Comment 2 mikhail.voytenko 2009-06-12 15:38:31 UTC
Created attachment 62947 [details]
The patch introducing missing changes.
Comment 3 mikhail.voytenko 2009-06-12 15:41:07 UTC
The problem was that some changes for issue 101219 was not commited.
The missing changes are commited in cws mav54_P1 now.
Comment 4 mikhail.voytenko 2009-06-12 15:41:59 UTC
mav->oc: please verify the issue after integration in the new master.
Comment 5 mikhail.voytenko 2009-06-12 16:30:49 UTC
Actually the problem could affect document of any format. In some circumstances
trying to write to a storage stream would throw exception, as result saving of
some documents might fail. Theoretically it is even possible that the document
is stored with information loss if application ignores the exception.

I am changing the summary accordingly.
Comment 6 wolframgarten 2009-06-15 11:45:50 UTC
*** Issue 102755 has been marked as a duplicate of this issue. ***
Comment 7 mikhail.voytenko 2009-06-18 11:45:25 UTC
The project is not directly mentioned in the attached diff, so just to simplify
the integration:
the patched in diff file is
comphelper/source/streaming/memorystream.cxx
Comment 8 oc 2009-06-30 12:40:23 UTC
verified in master DEV300_m51
Comment 9 oc 2009-06-30 12:41:09 UTC
closed because fix available in master
Comment 10 carsten.driesner 2009-07-01 09:25:04 UTC
*** Issue 103054 has been marked as a duplicate of this issue. ***
Comment 11 niklas.nebel 2009-07-02 17:08:53 UTC
*** Issue 103110 has been marked as a duplicate of this issue. ***
Comment 12 Joe Smith 2009-07-16 16:06:27 UTC
Verifying: I see correct behavior in DEV300_m52 (Linux)

Opened, saved and reloaded test document without error.
Comment 13 mhatheoo 2009-07-16 16:56:31 UTC
please have a look at issue 102275, which is quite similar to this one, and
which seems to be solved in m52 aswell 
(with m52 on WIN2K)

Please try to make this fix reach the 3.1.1-release

Martin