Good news and thanks Terry. So the bug would be closed automatically or, need I do anything further?
2012/6/29 YangTerry <polo8...@hotmail.com> > > > > > Test again on trunk r1355082 which include modules nss and moz. > From the result I think we can close this bug. > > Test with 3 scenario. > A. Saved with password, then remove password with same revision > B. Saved with password with OOo3.3, then remove password with r1355082 > C. Saved with password with MS 2003, then remove password with r1355082 > > Result: > Scenario A: > ods/xls/odt/odp work fine > doc can saved with password successfully, but when reopen the saved file, > it is under read-only mode. This also repro on OOo 3.3 > ppt is weird, in OOo 3.3, In saved dialog, the password check box can > check, input password and saved the file then reopen it, No need password, > so saved with password in OOo3.3 for ppt format is not work. > In trunk r1355082, In saved dialog, "saveing with password" checkbox is > disabled > If we saved ppt file with password protect by OpenOffice, then open in MS > Office, no need input password, so it is totally not work. > > Scenario B: > ods/xls/odt/odp work fine > doc file with password open under read-only mode, we can saved to another > file to remove password. > ppt file can't saved with password protect > > > Scenario C: > xls work fine > doc file with password open under read-only mode, we can saved to another > file to remove password. > ppt file with password protect by MS office can't open in Open Office, it > said "Read Error. the loading of password-encrypted Microsoft PowerPint > presentations is not supported." > > For PPT issue, there have 2 bugs about saved(Bug 39527) and loading(Bug > 46307) > For doc open with read-only mode issue, i think it is a know issue, but i > can't find the bug in bugzilla, I will double check in bugzilla. > > > > Date: Thu, 28 Jun 2012 11:20:54 +0200 > > From: orwittm...@googlemail.com > > To: ooo-dev@incubator.apache.org > > Subject: Re: Propose for 3.4.1: Can't remove password from file (119366) > > > > Hi, > > > > On 28.06.2012 11:08, Oliver-Rainer Wittmann wrote: > > > Hi, > > > > > > sorry for top posting - it is reply to the complete discussion > regarding "save > > > with password" > > > > > > This functionality for ODF documents is available when the build > includes > > > modules nss and moz. > > > > > > I am not sure, if our buildbots include these modules. I will check it. > > > > > > > win7 buildbot includes nss and moz > > linux32 buildbot includes nss and moz > > linux64 buildbot includes nss and moz > > > > But, as you may have notice since a couple of days our buildbots have > problems > > to create new packages. > > last sucessful build for win7 was on 2012-06-19 > > last sucessful build for linux32 was on 2012-06-20 > > last sucessful build for linxu64 was on 2012-06-17 > > none of these contains the fix for issue 119366 > > > > Best regards, Oliver. > > > > > > > > Best regards, Oliver. > > > > > > On 28.06.2012 10:21, YangTerry wrote: > > >> > > >> Just confirm with our build owner, moz package is not in the build. > > >> Will verify and update the result after download the trunk build from > BuildBot. > > >> > > >> Thanks Fan Zheng help (*^__^*) > > >> > > >> > > >>> From: polo8...@hotmail.com > > >>> To: ooo-dev@incubator.apache.org > > >>> Subject: RE: Propose for 3.4.1: Can't remove password from file > (119366) > > >>> Date: Thu, 28 Jun 2012 16:02:31 +0800 > > >>> > > >>> > > >>> Will confirm with our local build owner. Also download the trunk > build from http://ci.apache.org/projects/openoffice/ > > >>> > > >>> Thanks for your investgate. > > >>> > > >>>> Date: Thu, 28 Jun 2012 15:19:39 +0800 > > >>>> Subject: Re: Propose for 3.4.1: Can't remove password from file > (119366) > > >>>> From: zheng.easy...@gmail.com > > >>>> To: ooo-dev@incubator.apache.org > > >>>> > > >>>> Hold a second, you guys are using the download build on verifying, > right? > > >>>> > > >>>> If so, that means some problems there. Maybe the release build env > broken? > > >>>> > > >>>> > > >>>> 2012/6/28 Fan Zheng <zheng.easy...@gmail.com> > > >>>> > > >>>>> Muhaha, lucky for me that I have 3 build enviroment on > verification this, > > >>>>> all of them are under Windows XP. The difference are: > > >>>>> In build env A, there is no moz package involved; > > >>>>> In build env B, there is moz package involved, but without my > solution of > > >>>>> issue 119366; > > >>>>> In build env C, there is moz package involved and with my solution > of > > >>>>> issue 119366; > > >>> > > >>>>> And I did the following test cases: > > >>>>> In env A: > > >>>>> 1.1 Save into ODT, without password, passed; > > >>>>> 1.2 Save into ODT, with password, failed, with error message > "General > > >>>>> Error: Generral input/output error"; > > >>>>> 1.3 Save into DOC, without password, passed; > > >>>>> 1.4 Save into DOC, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> 2.1 Save into ODP, without password, passed; > > >>>>> 2.2 Save into ODP, with password, failed, with error message > "General > > >>>>> Error: Generral input/output error"; > > >>>>> 2.3 Save into PPT, without password, passed; > > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is > disabled... > > >>>>> Weird. > > >>>>> 3.1 Save into ODS, without password, passed; > > >>>>> 3.2 Save into ODS, with password, failed, with error message > "General > > >>>>> Error: Generral input/output error"; > > >>>>> 3.3 Save into XLS, without password, passed; > > >>>>> 3.4 Save into XLS, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> > > >>>>> In env B: > > >>>>> 1.1 Save into ODT, without password, passed; > > >>>>> 1.2 Save into ODT, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> 1.3 Save into DOC, without password, passed; > > >>>>> 1.4 Save into DOC, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> 2.1 Save into ODP, without password, passed; > > >>>>> 2.2 Save into ODP,with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> 2.3 Save into PPT, without password, passed; > > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is > disabled... > > >>>>> Weird. > > >>>>> 3.1 Save into ODS, without password, passed; > > >>>>> 3.2 Save into ODS, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> 3.3 Save into XLS, without password, passed; > > >>>>> 3.4 Save into XLS, with password, passed; But in continual case: > "Save As > > >>>>> without password", failed; > > >>>>> > > >>>>> In env C: > > >>>>> 1.1 Save into ODT, without password, passed; > > >>>>> 1.2 Save into ODT, with password, passed; In continual case: "Save > As > > >>>>> without password", Passed; > > >>>>> 1.3 Save into DOC, without password, passed; > > >>>>> 1.4 Save into DOC, with password, passed; In continual case: "Save > As > > >>>>> without password", Passed; > > >>>>> 2.1 Save into ODP, without password, passed; > > >>>>> 2.2 Save into ODP,with password, passed; In continual case: "Save > As > > >>>>> without password", Passed; > > >>>>> 2.3 Save into PPT, without password, passed; > > >>>>> 2.4 Save into PPT, the "saveing with password" checkbox is > disabled... > > >>>>> Weird. > > >>>>> 3.1 Save into ODS, without password, passed; > > >>>>> 3.2 Save into ODS, with password, passed; In continual case: "Save > As > > >>>>> without password", Passed; > > >>>>> 3.3 Save into XLS, without password, passed; > > >>>>> 3.4 Save into XLS, with password, passed; In continual case: "Save > As > > >>>>> without password", Passed; > > >>>>> > > >>>>> Conclution: > > >>>>> > > >>>>> Such error messages are caused missing moz package building > involved. > > >>>>> And such I/O error message only occurs in the ODF format file > saving with > > >>>>> password. > > >>>>> Both ODF format saving with password and OFFICE format saving with > > >>>>> password have the issue 119366; > > >>>>> My solution could work on both MS office files saving and ODF > format > > >>>>> saving; > > >>>>> > > >>>>> For you review. > > >>>>> > > >>>>> > > >>>>> 2012/6/28 YangTerry <polo8...@hotmail.com> > > >>>>> > > >>>>>> > > >>>>>> I can saved .ods file without password successfully, also can > save .xls > > >>>>>> successfully. > > >>>>>> > > >>>>>> Only when i saved our format (like .ods) file with password, this > error > > >>>>>> pop up. > > >>>>>> > > >>>>>> Also failed to open our format file with password. > > >>>>>> > > >>>>>> > > >>>>>> > > >>>>>> So it should not be related with build without moz package. > > >>>>>> > > >>>>>> > > >>>>>>> Date: Thu, 28 Jun 2012 13:57:14 +0800 > > >>>>>>> Subject: Re: Propose for 3.4.1: Can't remove password from file > (119366) > > >>>>>>> From: zheng.easy...@gmail.com > > >>>>>>> To: ooo-dev@incubator.apache.org > > >>>>>>> > > >>>>>>> And I my local, (WinXP env), seems such issues can not be > reproduced. > > >>>>>>> > > >>>>>>> > > >>>>>>> > > >>>>>>> 2012/6/28 Fan Zheng <zheng.easy...@gmail.com> > > >>>>>>> > > >>>>>>>> The error message "General Error: Generral input/output error" > looks > > >>>>>> so > > >>>>>>>> farmilliar... I remember that in the build without moz package > > >>>>>> inside, such > > >>>>>>>> dialog will show up. > > >>>>>>>> > > >>>>>>>> > > >>>>>>>> > > >>>>>>>> > > >>>>>>>> 2012/6/28 YangTerry <polo8...@hotmail.com> > > >>>>>>>> > > >>>>>>>>> > > >>>>>>>>> Verify not fixed on trunk r1354384. > > >>>>>>>>> If we saved with our format(.ods) > > >>>>>>>>> Failed to saved with password, the failed message in dialog is > > >>>>>> "General > > >>>>>>>>> Error: Generral input/output error", also failed open the > password > > >>>>>> protect > > >>>>>>>>> .ods file, it pop up password incorrect dialog but i input > correct > > >>>>>>>>> password. Reopen this bug. > > >>>>>>>>> My Platform is Win 7 X64 EN.If we saved with MS format (.xls), > > >>>>>>>>> successfully saved it with password and also work fine to > removed the > > >>>>>>>>> password. > > >>>>>>>>> Also work fine saved file(.ods) without password. > > >>>>>>>>> > > >>>>>>>>> Seems something wrong in our format save with password logic. > > >>>>>>>>> > > >>>>>>>>>> Date: Tue, 26 Jun 2012 16:27:56 +0200 > > >>>>>>>>>> From: orwittm...@googlemail.com > > >>>>>>>>>> To: ooo-dev@incubator.apache.org > > >>>>>>>>>> Subject: Re: Propose for 3.4.1: Can't remove password from > file > > >>>>>> (119366) > > >>>>>>>>>> > > >>>>>>>>>> Hi, > > >>>>>>>>>> > > >>>>>>>>>> On 26.06.2012 14:05, Oliver-Rainer Wittmann wrote: > > >>>>>>>>>>> Hi, > > >>>>>>>>>>> > > >>>>>>>>>>> On 26.06.2012 09:53, Fan Zheng wrote: > > >>>>>>>>>>>> Root cause: > > >>>>>>>>>>>> > > >>>>>>>>>>>> Seems the logic of "Save As" and "Save" inside Apache > > >>>>>> OpenOffice is > > >>>>>>>>> pretty > > >>>>>>>>>>>> weird anyway. > > >>>>>>>>>>>> A, inside AOO, the method SfxBaseModel::StoreSelf is the > entry > > >>>>>> for > > >>>>>>>>> storing > > >>>>>>>>>>>> file into the original URL path if it has one. Which means, > such > > >>>>>>>>> method is > > >>>>>>>>>>>> responsible to: > > >>>>>>>>>>>> > > >>>>>>>>>>>> 1. Directly "Save" request, but exclude the very first time > on > > >>>>>> "Save" > > >>>>>>>>>>>> without original URL path; > > >>>>>>>>>>>> > > >>>>>>>>>>>> 2. "SaveAs" request, with the same URL information as > former; > > >>>>>>>>>>>> > > >>>>>>>>>>>> > > >>>>>>>>>>>> B, as such method is only focus on storing back into to > original > > >>>>>>>>> file, it > > >>>>>>>>>>>> is designed as an incremental saving pattern for certain > > >>>>>> efficient > > >>>>>>>>>>>> consideration. Which means, > > >>>>>>>>>>>> such function do not allow external saving parameters > except the > > >>>>>>>>> ones on > > >>>>>>>>>>>> changing "Version Comments", "Author", "Interaction > Handler" and > > >>>>>>>>> "status > > >>>>>>>>>>>> Indicator". > > >>>>>>>>>>>> > > >>>>>>>>>>>> C, "Saving with password" is a kind of external saving > > >>>>>> parameter. The > > >>>>>>>>>>>> saving parameters set will contain a password item inside, > if > > >>>>>> users > > >>>>>>>>> have > > >>>>>>>>>>>> enable the check box > > >>>>>>>>>>>> "Save with password" in "File Save As" dialog. Otherwise, > saving > > >>>>>>>>> parameters > > >>>>>>>>>>>> set wont contain password corresponding items. > > >>>>>>>>>>>> > > >>>>>>>>>>>> Combine the above 3 conditions, we can take a deeper inside > > >>>>>> look of > > >>>>>>>>>>>> following scenarios: > > >>>>>>>>>>>> > > >>>>>>>>>>>> 1. In the "Save" request, whatever the password originally > > >>>>>> enabled > > >>>>>>>>> or not, > > >>>>>>>>>>>> as no further different setting applied, the storing process > > >>>>>> will > > >>>>>>>>> directly > > >>>>>>>>>>>> apply the former saving parameters set, including the URL > path > > >>>>>> and > > >>>>>>>>> password > > >>>>>>>>>>>> setting stuff. Everything is OK. > > >>>>>>>>>>>> > > >>>>>>>>>>>> 2. And in the "SaveAs" request with password originally > > >>>>>> disabled: > > >>>>>>>>>>>> 2.1 If the user keep the "Save with password" disabled in > "File > > >>>>>> Save > > >>>>>>>>> As" > > >>>>>>>>>>>> dialog, as no further setting applied, the storing process > will > > >>>>>>>>> directly > > >>>>>>>>>>>> apply the former saving parameters set, still with password > > >>>>>>>>> disabled. Keep > > >>>>>>>>>>>> the consistence between UI setting and exact result and high > > >>>>>>>>> efficiency; > > >>>>>>>>>>>> 2.2 If the user change the "Save with password" from > disable to > > >>>>>>>>> enable in > > >>>>>>>>>>>> "File Save As" dialog, as external saving parameter was > added > > >>>>>> into > > >>>>>>>>> saving > > >>>>>>>>>>>> parameters set, which do not satisfy the verification of > > >>>>>> parameters, > > >>>>>>>>> such > > >>>>>>>>>>>> "SaveAs" request will be returned from > SfxBaseModel::StoreSelf, > > >>>>>> and > > >>>>>>>>>>>> actually finished inside the common "SaveAs" method with > > >>>>>> password > > >>>>>>>>> enabled. > > >>>>>>>>>>>> Also keep the consistence between UI setting and exact > result; > > >>>>>>>>>>>> 3. In the "SaveAs" request with password originally > enabled: > > >>>>>>>>>>>> 3.1 If the user keep the "Save with password" enabled in > "File > > >>>>>> Save > > >>>>>>>>> As" > > >>>>>>>>>>>> dialog, as external saving parameter was added into saving > > >>>>>>>>> parameters set, > > >>>>>>>>>>>> which do not satisfy the verification of parameters, such > > >>>>>> "SaveAs" > > >>>>>>>>> request > > >>>>>>>>>>>> will be returned from SfxBaseModel::StoreSelf, and actually > > >>>>>> finished > > >>>>>>>>> inside > > >>>>>>>>>>>> the common "SaveAs" method with password enabled. Keep the > > >>>>>>>>> consistence > > >>>>>>>>>>>> between UI setting and exact result, but with lower > efficiency; > > >>>>>>>>>>>> 3.2 If the user change the "Save with password" from > enabled to > > >>>>>>>>> disabled in > > >>>>>>>>>>>> "File Save As" dialog, as no further setting applied, the > > >>>>>> storing > > >>>>>>>>> process > > >>>>>>>>>>>> will directly apply the former saving parameters set, still > with > > >>>>>>>>> password > > >>>>>>>>>>>> enabled, as oppose to the UI setting. The issue happens. > > >>>>>>>>>>>> > > >>>>>>>>>>>> So, a reasonable solution of this issue should be: > > >>>>>>>>>>>> > > >>>>>>>>>>>> 1. No process and saving parameter change on scenario 1 and > 2; > > >>>>>>>>>>>> 2. In scenario 3.1, remove the external password parameter > as > > >>>>>> the > > >>>>>>>>>>>> originally enabled, and makes it finished in StoreSelf for > > >>>>>> higher > > >>>>>>>>>>>> efficiency; > > >>>>>>>>>>>> 3. In scenario 3.2, do not trying to use StoreSelf anyway; > > >>>>>>>>>>>> > > >>>>>>>>>>>> > > >>>>>>>>>>>> For you reference. > > >>>>>>>>>>>> > > >>>>>>>>>>>> The code patch will be submitted for reviewing later. > > >>>>>>>>>>>> > > >>>>>>>>>>> > > >>>>>>>>>>> Thanks for this really deep and well founded analysis. > > >>>>>>>>>>> > > >>>>>>>>>>> I am currently reviewing the new patch. > > >>>>>>>>>>> > > >>>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>>> patch looks good - I will commit it to trunk and branch AOO34 > soon. > > >>>>>>>>>> Thx ZhengFan. > > >>>>>>>>>> > > >>>>>>>>>> Best regards, Oliver. > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>> > > >>>>>>>> > > >>>>>> > > >>>>>> > > >>>>> > > >>>>> > > >>> > > >> > > >> > > > > > > > >