Skip to Content.
Sympa Menu

grouper-dev - [grouper-dev] RE: [grouper] Implementation of missing parts of VOOT connector (#14)

Subject: Grouper Developers Forum

List archive

[grouper-dev] RE: [grouper] Implementation of missing parts of VOOT connector (#14)


Chronological Thread 
  • From: Andrea Biancini <>
  • To: Chris Hyzer <>
  • Cc:
  • Subject: [grouper-dev] RE: [grouper] Implementation of missing parts of VOOT connector (#14)
  • Date: Sat, 1 Nov 2014 17:46:29 +0100

Yes,
Probably my IDE changed indentatindentation...

Actually, however, if not any, most of the lines will be changed...

I will indento with 2 spaces instead of tab and send back the pull request.

Cheers,
Andrea

Il 01/nov/2014 12:42 "Chris Hyzer" <> ha scritto:

Andrea, thanks a lot for your pull request to fix the grouper-voot connector.

 

https://github.com/Internet2/grouper/pull/14/files

 

The diff in github says every line changed, can you please correct the indentation so it is the grouper standard 2 spaces and not tabs?  I wonder if that’s why github says every line changed, hopefully not a line ending issue too.  Do you mind correcting that and submitting another pull request so that the pull request says only changed lines have changed so its easier to diff and consistent with other Grouper code?  J

 

Thanks,

Chris

 

 

 

From: Andrea Biancini [mailto:]
Sent: Saturday, November 01, 2014 12:33 PM
To: Internet2/grouper
Subject: [grouper] Implementation of missing parts of VOOT connector (#14)

 

In this pull request I forward an update to the VOOT connector that, in respect to the previous implementation:

  • implements all the API endpoints of the VOOT protocol
  • implements the management of additional parameters (like search, startIndex, count, sortBy)
  • reviews code style, comments and some package name.

This development is still under-tested (since no JUnit has yet been created for it) however has been humanly tested on an existing Grouper 2.2.0 instance.


You can merge this Pull Request by running

  git pull https://github.com/biancini/grouper master

Or view, comment on, or merge it at:

  https://github.com/Internet2/grouper/pull/14

Commit Summary

  • Implemented all the handpoints of the VOOT protocol
  • Correct email type
  • Added pagination and sorting
  • Reviewed comments, removed unused code and eliminated magic strings
  • Reviewed comments, removed unused code and eliminated magic strings
  • Renamed message package
  • Renamed message package

File Changes

Patch Links:


Reply to this email directly or view it on GitHub.




Archive powered by MHonArc 2.6.16.

Top of Page