Apache OpenOffice (AOO) Bugzilla – Issue 121544
[AWT] Clean-up MessageBox API
Last modified: 2017-05-20 10:33:40 UTC
Besides the IDL errors described in bug 74095 the MessageBox API has some other shortcomings: a) css.awt.XMessageBox is deprecated, though it is returned by css.awt.XMessageBoxFactory.createMessageBox b) css.awt.MessageBoxCommand is deprecated, it can be removed c) the return type of css.awt.XMessageBox.execute() should be document with its own constants group d) css.awt.XMessageBoxFactory.createMessageBox() takes a css.awt.Rectangle defining the position and size of the message box, this is completely useless as it is ignored. The message box is always positioned in the center of its parent window, and the size is determined by the internal implementation e) the type of the message box in css.awt.XMessageBoxFactory.createMessageBox() is defined by strings, what makes it rather error prone. This should be replaced by an enumeration
"arielch" committed SVN revision 1425608 into trunk: i121544 - Clean-up MessageBox API
a) css.awt.MessageBoxCommand has been removed b) css.awt.MessageBoxResults has been introduced to specify a result of executing a css.awt.XMessageBox c) css.awt.MessageBoxType has been introduced to specify the type of a css.awt.XMessageBox on css.awt.XMessageBoxFactory.createMessageBox() d) in css.awt.XMessageBox the getters and setters have been replaced by interface attributes: CaptionText and MessageText e) css.awt.XMessageBoxFactory.createMessageBox() no longer takes a css.awt.Rectangle defining the position and size of the message box f) the type of the message box in css.awt.XMessageBoxFactory.createMessageBox() is no longer defined by strings, use css.awt.MessageBoxType instead
"arielch" committed SVN revision 1425715 into trunk: i121544 - Clean-up MessageBox API (post fix)
.
following files contains illegal value for preprocessor instructions: Index: offapi/com/sun/star/awt/MessageBoxType.idl =================================================================== --- offapi/com/sun/star/awt/MessageBoxType.idl (revision 1430225) +++ offapi/com/sun/star/awt/MessageBoxType.idl (working copy) @@ -19,8 +19,8 @@ * *************************************************************/ -#ifndef __com_sun_star_awt_MessageBoxCommand_idl__ -#define __com_sun_star_awt_MessageBoxCommand_idl__ +#ifndef __com_sun_star_awt_MessageBoxType_idl__ +#define __com_sun_star_awt_MessageBoxType_idl__ module com { module sun { module star { module awt { Index: offapi/com/sun/star/awt/MessageBoxResults.idl =================================================================== --- offapi/com/sun/star/awt/MessageBoxResults.idl (revision 1430225) +++ offapi/com/sun/star/awt/MessageBoxResults.idl (working copy) @@ -19,8 +19,8 @@ * *************************************************************/ -#ifndef __com_sun_star_awt_MessageBoxCommand_idl__ -#define __com_sun_star_awt_MessageBoxCommand_idl__ +#ifndef __com_sun_star_awt_MessageBoxResults_idl__ +#define __com_sun_star_awt_MessageBoxResults_idl__ module com { module sun { module star { module awt {
"arielch" committed SVN revision 1487143 into trunk: i121544 - Fix header guards
(In reply to hanya from comment #5) > following files contains illegal value for preprocessor instructions: IIRC you opened a new bug for this one, but I cannot find it right now. Please resolve it as fixed or make it depend on this one.
(In reply to Ariel Constenla-Haile from comment #7) > (In reply to hanya from comment #5) > > following files contains illegal value for preprocessor instructions: > > IIRC you opened a new bug for this one, but I cannot find it right now. > Please resolve it as fixed or make it depend on this one. Done in issue 121951. Thanks.