{"comment":{"position":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/4326d5e4ba2ddde64eecbb2bcb3e3f57?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"4326d5e4ba2ddde64eecbb2bcb3e3f57","url":"https://api.github.com/users/Toasty27","login":"Toasty27","id":745524},"updated_at":"2011-11-08T10:17:25Z","commit_id":"3899adb474a7139142cdd9f720d618c71f079453","line":null,"url":"https://api.github.com/repos/Toasty27/codecraft-init-script/comments/702210","body":"There is a potential problem with this script due to it's use of time and dates. There is the potential that the directories could be mis-named if the backup function is called immediately prior to midnight (say, 11:59:59 or something). In that event, the current code would generate a folder titled 00:0X (where X = some integer) under the folder with the date of the previous day. \r\n\r\nTo fix this, a check needs to be made after the creation of the child directory to verify that the current date matches that of the label given to the parent directory. In the event that it doesn't match, the child directory needs to be moved to a folder who's label is the current date (as given by `date \"+%Y.%m.%d\"` ).\r\n\r\nThis needs to be done to prevent a folder from being made that has a label suggesting it's older than another folder in the same directory when it is, in fact, about 24 hours older.","html_url":"https://github.com/Toasty27/codecraft-init-script/commit/3899adb474#commitcomment-702210","id":702210,"created_at":"2011-11-08T10:17:25Z","path":null}}