Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requesting change of the access modifiers in RDNodeResource #25

Open
navoda opened this issue Oct 18, 2016 · 4 comments
Open

Requesting change of the access modifiers in RDNodeResource #25

navoda opened this issue Oct 18, 2016 · 4 comments

Comments

@navoda
Copy link

navoda commented Oct 18, 2016

Hi,
There are some private methods that I need to extend and override in RDNodeResource class for project specializing purposes.

Eg: The complete request payload is handled by the private method updateEndpointResources, which makes it impossible to extract any additional information from the payload.

Can you change them to public or any other access modifiers that can be override?

@sophokles73
Copy link
Contributor

@navoda,

is the stuff that is currently done in updateEndpointResources all wrong and you need to do something completely different or do you just need an extension hook that will be called from there and which you can implement by subclassing? I am asking because I would like to better understand which functionality should be provided by RDNodeResource and what kind of extensibility is required..

@navoda
Copy link
Author

navoda commented Oct 20, 2016

The whole RDNodeResource class should be changed into more generalized mode.

I will describe a scenario briefly, for you to understand the matter;

The RD receives a registration request for a specific endpoint with the resource list to add with their resource types.
Currently RDNodeResource adds these resources to the endpoint as sub resources (which is fine) but all the resource types are hard-coded to be instances of RDTagResource -> This is done by the function addNodeResources.
But I need those resources to be various types of resources with-respect to their rt (resource type) attribute.
But the attributes are set by the function updateEndpointResources only after executing addNodeResources which already add resources as 'new RDTagResources'. Making it impossible to make the changes afterward.

I think this is actually an issue of process order and poor generalization of the RDNodeResource class. But I can get a temporary solution for my project, if the class is more flexible for override facility in a subclass.

I would really like if I can send a PR with a better solution for the matter. But right now I'm facing some issues with signing a CLA.

@sophokles73
Copy link
Contributor

Can you give me your email address that you used for your Eclipse account? I will try to reach out to staff there I know regarding this problem ...

@navoda
Copy link
Author

navoda commented Oct 21, 2016

@sophokles73
mpnavoda@gmail.com, this is the same email I use for GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants