Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 336816

Summary:

Easier access of navigation arguments

Product:

[RT] Riena

Reporter:

Stephan Mann <stephan.mann>

Component:

navigation

Assignee:

Project Inbox <riena.core-inbox>

Status:

NEW ---

QA Contact:

Severity:

enhancement

Priority:

CC:

christian.campo

Version:

2.0.0

Target Milestone:

---

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
convenience methods for navigation	none

Description Stephan Mann

2011-02-10 08:20:48 EST

As navigation is an important part in Riena, using it could be easier. Many controllers need to access navigation arguments or their parameters or need to navigate to another controller. During a customer project we have come up with 5 convenience methods, that make life easier and prevent the developer from writing the same code over and over again. 

The attached patch adds these methods to SubModuleController. I'm not sure whether they'd better be moved to NavigationNode.

Comment 1 Stephan Mann

2011-02-10 08:21:43 EST

Created attachment 188680 [details]
convenience methods for navigation

Comment 2 Christian Campo

2011-05-30 02:58:41 EDT

how about a declarative approach ?

@RequiresNavigationParameter(type=Customer.class)
public void configureRidgets(Customer customer) {
}

Comment 3 Stephan Mann

2011-05-30 07:39:48 EDT

The declarative approach would be preferable, but IMHO not at the cost of an impaired flexibility. So multiple annotations would be required to differentiate between required and optional navigation parameters.

Comment 4 Christian Campo

2011-05-30 08:34:01 EDT

(In reply to comment #3)
> The declarative approach would be preferable, but IMHO not at the cost of an
> impaired flexibility. So multiple annotations would be required to
> differentiate between required and optional navigation parameters.

I totally agree. I was also thinking of multiple annotations. (just too lazy to actually write them down :-) )