25 Aug, 2015

12 commits

  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12788 41b1ffd8-f28e-4786-ab96-9950f0a78031
    fabiop
     
  • Used Collection.contains().
    
    LDAPURL.java:
    Simplified equals().
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12787 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Used early exits
    Removed duplicated code
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12786 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Changed getNormalizedNames() to return a Set instead of Iterable.
    
    *.java:
    Consequence of the change to CommonSchemaElements.getNormalizedNames().
    Used early exits.
    Extracted method SchemaElementPanel.getAliases().
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12785 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • In mainVerifyIndex():
    - use Collection.contains()
    - release the shared lock in a finally clause
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12784 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12783 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12782 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12781 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12780 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • 
    Entry.java:
    In applyModification(), extracted methods applyModificationToObjectclass() and applyModificationToNonObjectclass().
    In hasUserAttribute() and hasOperationalAttribute(), extracted method hasAttribute()
    Used Attribute.isReal() to explicit code + merged if statements.
    
    
    Attribute.java:
    Added isReal() -> !isVirtual().
    
    AbstractAttribute.java:
    Implemented isReal().
    In equals() extracted method valuesEqual().
    In hasAllOptions() and optionsEqual(), extracted method hasAllOptions0().
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12779 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • * org.opends.server.protocols.http.HttpConnectionHandler
    ** Fix copyright section
    ** Make code more readable
    *** Reformat
    *** Inline some local variables
    
    * org.opends.server.protocols.http.HTTPClientConnection
    ** Fix copyright section
    ** Reformat
    
    * org.opends.server.protocols.http.CollectClientConnectionsFilter
    ** Fix copyright
    
    * org.opends.server.protocols.http.CollectClientConnectionsFilterTest
    ** Reformat
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12778 41b1ffd8-f28e-4786-ab96-9950f0a78031
    gaetan
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12777 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     

24 Aug, 2015

5 commits


21 Aug, 2015

7 commits

  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12771 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • 
    NamingConflictTest.java
    Extracted fields to simplify code in test methods.
    Extracted methods setUpLocal(), tearDown(), addMsg(), modDnMsg(), replayMsg(), createAndAddEntry(), createParentEntry(), createChildEntry().
    
    ModifyConflictTest.java:
    Made the code a bit more readable.
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12770 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Extracted methods replaceOrDelete(), add(), replace(), delete() and deleteWithoutDeleteTime() to increase code expressiveness.
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12769 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Used ReplicationTestCase.waitForSpecificMsg() and waitForSpecificMsgs() to ignore unrelated messages.
    
    ReplicationTestCase.java:
    Added methods:
    - waitForSpecificMsgs(Session session, Class<?>... msgTypes)
    - waitForSpecificMsgs(ReplicationBroker broker, Class<?>... msgTypes)
    Changed
    <T extends ReplicationMsg> T waitForSpecificMsg(Session session, ReplicationBroker broker, Class<T> msgType)
    to
    ReplicationMsg waitForSpecificMsgs(Session session, ReplicationBroker broker, Class<?>... msgTypes)
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12768 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Added test values.
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12767 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Improved toString()
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12765 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Tests methods were not properly isolated. The sequence testUpdateDeletedIDs() followed by testUpdateAddedIDs() caused the following failure:
    17:42:32                  T E S T   F A I L U R E ! ! !
    17:42:32 
    17:42:32 Failed Test:  org.opends.server.backends.pluggable.DefaultIndexTest#testUpdateAddedIDs
    17:42:32 Failure Cause:  java.lang.AssertionError: 
    17:42:32 Expecting:
    17:42:32  <[]>
    17:42:32 to contain:
    17:42:32  <[1, 2, 3, 4]>
    17:42:32 but could not find:
    17:42:32  <[1, 2, 3, 4]>
    17:42:32 
    17:42:32     org.opends.server.backends.pluggable.Utils.assertIdsEquals(Utils.java:39)
    17:42:32     org.opends.server.backends.pluggable.Utils.assertIdsEquals(Utils.java:51)
    17:42:32     org.opends.server.backends.pluggable.DefaultIndexTest.testUpdateAddedIDs(DefaultIndexTest.java:76)
    
    
    
    DefaultIndexTest.java:
    In setUp(), use @BeforeMethod annotation (rather than @BeforeTest)
    
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12764 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     

20 Aug, 2015

4 commits


19 Aug, 2015

3 commits


18 Aug, 2015

3 commits

  • In processLocalOrNonConflictModification(), made the code more explicit + prevented double assignment
    
    *.java:
    Added toString().
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12755 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • 
    AttrHistorical.java:
    In getValuesHistorical(), changed return type from Map<AttrValueHistorical, AttrValueHistorical> to Set<AttrValueHistorical> - to match the usage.
    
    AttrHistoricalMultiple.java:
    Consequence of the change to AttrHistorical.getValuesHistorical().
    In delete(Attribute, CSN), now call delete(ByteString, CSN).
    In add(Attribute, CSN), now call add(ByteString, CSN).
    Extracted method update(CSN, AttrValueHistorical).
    In conflictAdd(), changed return type to void since it was never used.
    Removed javadoc copied from supertype + useless @inheritDoc.
    Reduced method's visibilities.
    
    AttrHistoricalSingle.java
    Consequence of the change to AttrHistorical.getValuesHistorical().
    Removed javadoc copied from supertype + useless @inheritDoc.
    Used static import for HistAttrModificationKey.*.
    Extracted method getSingleValue().
    
    EntryHistorical.java:
    Consequence of the change to AttrHistorical.getValuesHistorical().
    Extracted method toOptionsString().
    In newInstanceFromEntry(), reduced scope of variables.
    
    AttrInfoTest.java:
    Consequence of the change to AttrHistorical.getValuesHistorical().
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12754 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • …th the more efficient Utils.joinAsString(StringBuilder, String, Collection).
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12753 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     

17 Aug, 2015

6 commits

  • Removed fields duplicating the ones from CoreSchema + replaced them with the getters directly delegating to CoreSchema methods.
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12752 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Remove class which has been moved to org.opends.server.backends.jeb
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12751 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • In assertChangelogAttributesInRootDSE(), removed always true parameter "isECLEnabled".
    
    ReplicationTestCase.java:
    Code cleanup
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12750 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12747 41b1ffd8-f28e-4786-ab96-9950f0a78031
    cjr
     
  • git-svn-id: https://svn.forgerock.org/opendj/trunk@12746 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac
     
  • Fix a bug in RootDSEBackend.getRootDSE().
    
    RootDSEBackend.java:
    In getRootDSE(), fix a bug with supportedAuthPWSchemes which wrongly referred to supportedSASLMechanism attribute.
    In addAttribute(), renamed publicNamingContextAttr parameter to attribute
    Used interfaces for collections rather than concrete classes.
    Removed useless @inheritDoc javadoc.
    
    Task.java:
    Extracted methods getTime(), compareTimes(), 
    In execute(), simplified code.
    
    LDAPReplicationDomain.java, ReplicationDomainMonitorData.java:
    Code cleanups
    
    git-svn-id: https://svn.forgerock.org/opendj/trunk@12745 41b1ffd8-f28e-4786-ab96-9950f0a78031
    JnRouvignac