Issue 92508 - [api] css.sdb.application.DefaultViewController does not what it promises to do ...
Summary: [api] css.sdb.application.DefaultViewController does not what it promises to ...
Status: CLOSED FIXED
Alias: None
Product: Base
Classification: Application
Component: code (show other issues)
Version: OOo 3.0 Beta 2
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 3.0
Assignee: Frank Schönheit
QA Contact: issues@dba
URL:
Keywords: crash
Depends on:
Blocks: 88888
  Show dependency tree
 
Reported: 2008-08-06 10:25 UTC by Frank Schönheit
Modified: 2017-05-20 10:27 UTC (History)
1 user (show)

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


Attachments
patch (modified version of Ariel's patch) (18.83 KB, patch)
2008-08-06 10:26 UTC, Frank Schönheit
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description Frank Schönheit 2008-08-06 10:25:56 UTC
The service css.sdb.application.DefaultViewController, which has been newly
introduced with OOo 3.0, and effectively describes the view/controller API for
the Base application, implements the two interfaces XContextMenuInterception and
XSelectionSupplier. Those interfaces are highly valuable for extensions which
want to hook into the database application UI, and extend it with own functionality.

Unfortunately, as Ariel Constenla-Haile found out and reported in dev@dba
(http://dba.openoffice.org/servlets/BrowseList?list=dev&by=thread&from=2099228),
both interfaces have bugs to a degree which makes them nearly useless:
The XSelectionChangeListener usage crashes OOo when you invoke it at the "wrong"
time, and the XContextMenuInterception implementation loses important
information of the extension-provided context menus, such as images, and breaks
separators in the context menu.


To show the problems, you can use the extension provided by Ariel, to be found
here:
http://www.ArielConstenlaHaile.com.ar/ooo/tmp/dbaccess/DatabaseControllerDemo.zip

Also, Ariel already provided a patch:
http://dba.openoffice.org/servlets/ReadMsg?list=dev&msgNo=3524
Comment 1 Frank Schönheit 2008-08-06 10:26:44 UTC
Created attachment 55592 [details]
patch (modified version of Ariel's patch)
Comment 2 Frank Schönheit 2008-08-06 10:56:55 UTC
The crash when you install the extension in Base justifies a P2, /me thinks.
Comment 3 Frank Schönheit 2008-08-06 15:10:08 UTC
fixed in CWS reportbuilderext

find more information about this CWS, like when it is available in the master
builds, in EIS, the Environment Information System:
http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Path=DEV300%2Freportbuilderext