Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#1554 closed enhancement (fixed)

Add support for extending multiple extension points at the same time

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: BASE 2.17
Component: web Version:
Keywords: Cc:

Description (last modified by Nicklas Nordborg)

The current extension system allows a single extension to extend a single extension point. But we also define some generic extension points (eg. net.sf.basedb.clients.web.toolbar.list.all) which makes it possible for extensions to appear in more than one place.

But if none of the generic extension points are suitable an extension must be defined multiple times to be used on multiple extension points. It would be better to allow a single extension to extend multiple extension points in the first place.

My idea is that as an alternative to specify the extension point as an attribute in <extension id="..." extends="..."> we should support something like this:

<extension id="..">
  <extends>
    <ref index="..">....</ref>
    <ref index="..">....</ref>
  </extends>
  ...
</extension>

The rest of the extension definition remains the same and it is of course assumed that all given extension points are using a compatible Action.

Note that this is a change to the XML format only. Internally multiple extension objects will be created.

Change History (3)

comment:1 by Nicklas Nordborg, 13 years ago

Owner: changed from everyone to Nicklas Nordborg
Status: newassigned

comment:2 by Nicklas Nordborg, 13 years ago

Resolution: fixed
Status: assignedclosed

(In [5519]) Fixes #1554: Add support for extending multiple extension points at the same time

comment:3 by Nicklas Nordborg, 13 years ago

Description: modified (diff)

Changed the XML example to show how it was actually implemented.

Note: See TracTickets for help on using tickets.