graphaware / neo4j-framework

GraphAware Neo4j Framework
244 stars 68 forks source link

Decrease coupling to Spring / Cypher. #45

Closed jotomo closed 8 years ago

jotomo commented 8 years ago

Scenario:

This commit removes imports of StringUtils from parboiled (transitive dep from Cypher), I'm assuming these are accidential. I have replaced them with calls to commons-lang's StringUtils.

Then there are some usages of Spring's Assert helper class, mostly the notNull method. I replaced these calls with Objects.requireNonNull (a not widely known method introduced in Java 7).

These changes allow very selectively depeding on just three graphaware modules (excluding all transitive dependencies) without having to bring in Cypher or Spring.

This PR is against the 3.0 branch. If you prefer a PR against master to forward-merge from let me know.