Bug Report #3032
Session's constructor shouldn't be protected
| Status: | Closed | Start date: | 06/29/2010 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Woody Gilk |
% Done: | 0% |
|
| Category: | Core | |||
| Target version: | v3.0.7 | |||
| Resolution: | fixed | Points: |
Description
The class is already abstract, having a protected constructor just makes it harder to test.
I've implemented this in my fork and sent a pull request
Associated revisions
Modifying Session to use public constructor. Class is already abstract,
this makes it easier to test.
Fixes #3032
History
Updated by Matt Button almost 3 years ago
Updated by Woody Gilk almost 3 years ago
- Status changed from New to Closed
- Target version changed from v3.0.6 to v3.0.7
- Resolution set to fixed