We're updating the issue view to help you get more done. 

Needless code in AuthorizationManager

Description

The AuthorizationManager has a getInstance() method and private constructor, which makes it a singleton.

Unfortunately, it has only static data and methods so the creation of any instance is completely pointless. It is a utility class (i.e. static), and should not be singleton. The getInstance() method and instance member should be removed.

Environment

None

Acceptance Test - Entry

None

Activity

Show:
Tom Evans
April 25, 2014, 7:56 PM

Agreed, and is a quick fix. I found no usages of the extraneous getInstance() method.

Tom Evans
April 25, 2014, 11:23 PM

Removed getInstance(), instance variable, and constructor.

Assignee

Tom Evans

Reporter

Robin Collier

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Trivial
Configure