Issue 94938 - unopkg: failure to show any installed extensions if one is mis-installed
Summary: unopkg: failure to show any installed extensions if one is mis-installed
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m32
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.1
Assignee: joerg.skottke
QA Contact: issues@framework
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-15 10:27 UTC by caolanm
Modified: 2009-02-24 14:35 UTC (History)
1 user (show)

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

Attachments
patch to workaround (816 bytes, patch)
2008-10-15 10:28 UTC, caolanm 		no flags Details | Diff
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 caolanm 2008-10-15 10:27:00 UTC 
If for some reason the installation or de-installation of an individual
extension fails to complete correctly, then attempts to get a list of installed
extensions will throw a deployment::DeploymentException exception, which causes
lots of problems, e.g. see https://bugzilla.redhat.com/show_bug.cgi?id=466881

I see that we already catch one type of exception when trying to iterate through
the extensions to find what is installed, so attached is a patch to catch this
one too so that unopkg and friends can keep on trucking and just ignore the
failed extension and return the rest of them successfully
Comment 1 caolanm 2008-10-15 10:28:01 UTC 
Created attachment 57142 [details]
patch to workaround
Comment 2 Mathias_Bauer 2008-10-16 09:31:01 UTC 
How did you manage to submit an issue in the last days? ;-)
Web site now is back again. 

Thanks for the patch, I forward it to the responsible engineer. As we found out
in the release phase the exception handling in the extension manager needs some
considerations anyway.
Comment 3 joachim.lingner 2008-10-21 11:17:20 UTC 
.
Comment 4 joachim.lingner 2009-01-15 16:05:20 UTC 
applied
Comment 5 joachim.lingner 2009-01-19 11:40:34 UTC 
@jsk: I have no good idea how to test this. Please just run your automated
extension tests.
Comment 6 joerg.skottke 2009-01-28 10:58:53 UTC 
Done, results are in sync with MWS. Verified.
Comment 7 caolanm 2009-02-24 14:35:20 UTC 
closed, seen DEV300_m42 and OOO310_m1