|
Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
| Summary: | suppress second error message for 'not existing license file' | ||
|---|---|---|---|
| Product: | udk | Reporter: | b.osi.ooo |
| Component: | code | Assignee: | AOO issues mailing list <issues> |
| Status: | ACCEPTED --- | QA Contact: | |
| Severity: | Trivial | ||
| Priority: | P3 | CC: | issues |
| Version: | OOo 1.0.0 | ||
| Target Milestone: | 4.x | ||
| Hardware: | All | ||
| OS: | Windows, all | ||
| Issue Type: | DEFECT | Latest Confirmation in: | --- |
| Developer Difficulty: | --- | ||
. getLicenseText in desktop/source/deployment/registry/package/dp_package.cxx should throw a CommandFailedException when descContent.openStream(); fails. This indicates that the error has already been handled by the InteractionHandler (see DialogImpl::SyncPushButton::Click() in D:\work\cws-jl40\desktop\source\deployment\gui\dp_gui_dialog.cxx) I guess it is better to leave the exception as it is. All error messages which have to be displayed to the user, either in the console or through dialogs, have be done through an interaction. So the message that comes from SyncPushButton::Click() is an additional message and should not displayed in pro builds. . . verified by autotest backed out again because other error messages were supressed. . . . set target to 3.x according to http://wiki.services.openoffice.org/wiki/Target_3x |
How to reproduce: add via extension manager: (from OOo cvs) qa/qatesttool/framework/basic/input/extensions/simpleLicense\MissinLic.oxt after the message: Adding Package(s): $FILE does not exist. a second ulgy one comes up: (com.sun.star.deployment.DeploymentException) { { Message = "Could >>> not read license text in >>> file:///C:/Documents%20and... the ugly one could be supressed?