GrailsInAction / graina2

Source code for the 2nd edition of Grails in Action
90 stars 93 forks source link

Suggestion for reader changes to listing 8.5 is confusing #131

Open pledbrook opened 9 years ago

pledbrook commented 9 years ago

The paragraph immediately after listing 8.5 suggests that the reader may want to try testing "minutes ago" and "seconds ago" as an exercise. That's fine for the former, but the current implementation of getNiceDate() doesn't handle "seconds ago". The implication is that the reader only has to work on the test case, but in fact they need to modify the getNiceDate() implementation as well.