EHC ❯ CacheConfiguration set*() methods could return this for chaining as well

• Type: Bug
• Status: Closed
• Priority: 2 Major
• Resolution: Won't Fix
• Component/s: ehcache-core
• Labels:

• Assignee: drb
• Reporter: amiller
• Created: April 05, 2010
• Votes: 0
• Watchers: 1
• Updated:

  July 27, 2012
• Resolved:

  April 13, 2010

Description

I tried to use the fluent api on CacheConfiguration and I accidentally did:

new Cache(new CacheConfiguration(“foo”, 100).setEternal(true)))

However, setEternal() doesn’t return “this” so can’t be chained. It would be nice if the set methods did return “this” so they could be chained OR if the javadoc for setEternal() referred you to eternal() which you are less likely to immediately notice.

The following code does work fine as intended:

new Cache(new CacheConfiguration(“foo”, 100).eternal(true)))

Comments

Geert Bevin 2010-04-13

This isn’t possible since it breaks the javabeans spec and those methods aren’t detected as setters anymore, they sadly absolutely need to return void.