directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shawn McKinney (JIRA)" <>
Subject [jira] [Commented] (FC-135) Rename ambigous getter and setters names on User and UserAdminRole
Date Sat, 09 Jan 2016 00:12:39 GMT


Shawn McKinney commented on FC-135:

Pull Request received:

GitHub user pike1212 opened a pull request:

   Rename Set Methods to avoid Conflict

   I renamed the duplicate set methods. My junit tests don't work even before I made the change,
so not sure how to propertly test. This is the error I get when running "mvn test -Dtest=FortressJUnitTest"

   Tests run: 141, Failures: 5, Errors: 0, Skipped: 0, Time elapsed: 228.481 sec <<<
   testSearchBinds(  Time elapsed:
0.022 sec  <<< FAILURE!
failed search for successful authentication user [jtsUser1]
   	at junit.framework.Assert.assertTrue(
   	at junit.framework.TestCase.assertTrue(

You can merge this pull request into a Git repository by running:

   $ git pull master

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

   This closes #4

commit 1e664a2e49b65c33e323e43beba88c0816601076
Author: clp207 <>
Date:   2016-01-05T17:01:31Z

   renamed set methods with same name in models

commit 417dc4d23858a059306f3a1d5b3f0588e71f74f3
Author: clp207 <>
Date:   2016-01-05T18:12:31Z

   changed method name to have List isntead of 's'

commit eae93c8ff3eb1f2b7e556ff35191a06fa4b30564
Author: clp207 <>
Date:   2016-01-05T18:21:14Z

   fixed test classes


> Rename ambigous getter and setters names on User and UserAdminRole
> ------------------------------------------------------------------
>                 Key: FC-135
>                 URL:
>             Project: FORTRESS
>          Issue Type: Improvement
>    Affects Versions: 1.0.0-RC40
>            Reporter: Shawn McKinney
>             Fix For: 1.0.0-RC41
> Ticket was a request from Chris Pike:
> On Jan 5, 2016, at 10:35 AM, Chris Pike <> wrote:
> There are a couple instances where models have multiple set methods for the same field.

> UserAdminRole has...
>    public void setOsP( String osP )
>    public void setOsP( Set<String> osPs )
>    public void setOsU( Set<String> osUs )
>    public void setOsU( String osU )
> User has...
>    public void setRole( String roleName )    
>    public void setRole( UserRole role )
>    public void setAdminRole( UserAdminRole role )
>    public void setAdminRole( String roleName )
> This is causing some issues down the line with jackson. I have a workaround but wanted
to see if there was any possibility of changing one of the method names.

This message was sent by Atlassian JIRA

View raw message