User class should not depend on Database class when driver implements internal methods #38

Closed
opened 2017-02-19 00:40:50 -05:00 by jking · 2 comments
Owner

The User class incorrectly interfaces directly with the Database class even when the driver reports implementing a method internally. This occurs with e.g. the exists() method; conseqently the first test in e6feb8de8d is wrong and the whole series needs to be fixed by not creating $this->data->db.

The User class incorrectly interfaces directly with the Database class even when the driver reports implementing a method internally. This occurs with e.g. the exists() method; conseqently the first test in e6feb8de8dc9b0c2f4357e9a60920e1de9b6b0b2 is wrong and the whole series needs to be fixed by not creating $this->data->db.
Author
Owner

This should be fixed by commit 7785eb072b, but there may still be bugs; test coverage needs to improve before this issue can be closed.

This should be fixed by commit 7785eb072ba927aa1ccc8d63425fef3b479e4cc2, but there may still be bugs; test coverage needs to improve before this issue can be closed.
Author
Owner

Confirmed fixed by commit 1834cb9963.

Confirmed fixed by commit 1834cb9963262cb164ff3f9d252fdf12d7dcfa51.
jking closed this issue 2017-02-25 13:01:12 -05:00
jking added this to the 0.1.0 milestone 2017-08-30 08:31:25 -04:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: MensBeam/Arsse#38
No description provided.