Kuupäeva võrdluse parandus

[varres]

aktiivne on kirje ka siis, kui kuupäev on
homme, mitte ainult aastal 9999

Sulgeja ja suletud võib teha ühe updatega, ei ole mõtet kiusata baasi
Query q = entityManager().createQuery("UPDATE " + table + " SET suletud=:date, sulgeja=:sulgeja WHERE id=" + id);
q.setParameter("date", getDate());
q.setParameter("sulgeja", user);

Eemaldatud koodi dubleerimine. Kui if'is ja elses on lõpus sama funktsionaalsus, siis see võib olla peale if'i

[siimav]

Kuupäevade võrdluse muutmine on täiesti mõttetu minu arvates. Ainus, mis selle muudatusega saavutataks oleks see, et saaks märkida mingi kirje kustutatuks alles kunagi tulevikus. Paraku ei näe aga sellel absoluutselt mingit mõtet.
Teine commit on igati asjalik ja oleksin nõus selle meie projekti mergema. Tegelikult võiks keegi veel transactionid ka tööle panna. :)

[varres]

Kuupäevadega selline lugu, et tegu on minuarust üsna suure loogika veaga. Hetkel see vast toimib, kuid ei tea kunagi ette, mida klient järgmisena soovib.. mõttetu on endale tulevikuks lisatööd teha, kui võimalik kohe korrektselt teha. Jään endiselt oma arvamuse juurde