Skip to content

getAllPlayerStatistics error vs empty ambiguity #19

Description

@DiamondDagger590

Description

getAllPlayerStatistics() returns an empty map on SQLException, making DB errors indistinguishable from "no data found". This can cause data loss if a connection failure during player load results in populateFromEntries({}) followed by markClean().

Solution

Either throw a checked exception, return a result wrapper, or use a flag to distinguish error from empty.

Files

  • PlayerStatisticDAO.java

Priority: Medium

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions