{"comment":{"url":"https://api.github.com/repos/commercetools/sphere-donut/comments/13888653","html_url":"https://github.com/commercetools/sphere-donut/commit/86f71fcec2ddc6d17751a9b0354f46877a628df1#commitcomment-13888653","id":13888653,"user":{"login":"lauraluiz","id":1917621,"avatar_url":"https://avatars.githubusercontent.com/u/1917621?v=3","gravatar_id":"","url":"https://api.github.com/users/lauraluiz","html_url":"https://github.com/lauraluiz","followers_url":"https://api.github.com/users/lauraluiz/followers","following_url":"https://api.github.com/users/lauraluiz/following{/other_user}","gists_url":"https://api.github.com/users/lauraluiz/gists{/gist_id}","starred_url":"https://api.github.com/users/lauraluiz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lauraluiz/subscriptions","organizations_url":"https://api.github.com/users/lauraluiz/orgs","repos_url":"https://api.github.com/users/lauraluiz/repos","events_url":"https://api.github.com/users/lauraluiz/events{/privacy}","received_events_url":"https://api.github.com/users/lauraluiz/received_events","type":"User","site_admin":false},"position":140,"line":118,"path":"app/services/CartServiceImpl.java","commit_id":"86f71fcec2ddc6d17751a9b0354f46877a628df1","created_at":"2015-10-20T23:00:23Z","updated_at":"2015-10-20T23:00:23Z","body":"I wouldn't access directly any `nullable` field (also it leaves you an ugly warning :)). Always surround it by `Optional.ofNullable` or a similar approach. Otherwise in this case, you will see a not nice NPE when an old cart without custom field that someone has in its session is fetched."}}