{"action":"created","comment":{"url":"https://api.github.com/repos/exodev/ecms/pulls/comments/89762845","pull_request_review_id":10297985,"id":89762845,"diff_hunk":"@@ -42,9 +44,12 @@ public UINodeInfo() {\n   }\r\n \r\n   public void update() throws Exception {\r\n-    UIVersionInfo uiVersionInfo = getAncestorOfType(UIDocumentWorkspace.class).getChild(UIVersionInfo.class) ;\r\n+    UIVersionInfo uiVersionInfo = getAncestorOfType(UIDocumentWorkspace.class).getChild(UIVersionInfo.class);\r\n     node_ = NodeLocation.getNodeLocationByNode(uiVersionInfo.getCurrentNode());\r\n-    versionCreatedDate_ = uiVersionInfo.getCurrentVersionNode().getCreatedTime().getTime().toString();\r\n+    Calendar createdTime = uiVersionInfo.getCurrentVersionNode().getCreatedTime();\r\n+    if (createdTime != null) {\r","path":"core/webui-explorer/src/main/java/org/exoplatform/ecm/webui/component/explorer/versions/UINodeInfo.java","position":18,"original_position":18,"commit_id":"bbacb43900ebfcc6dd3a64bdff37f90ea2999913","original_commit_id":"bbacb43900ebfcc6dd3a64bdff37f90ea2999913","user":{"login":"thomasdelhomenie","id":342645,"avatar_url":"https://avatars.githubusercontent.com/u/342645?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasdelhomenie","html_url":"https://github.com/thomasdelhomenie","followers_url":"https://api.github.com/users/thomasdelhomenie/followers","following_url":"https://api.github.com/users/thomasdelhomenie/following{/other_user}","gists_url":"https://api.github.com/users/thomasdelhomenie/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasdelhomenie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasdelhomenie/subscriptions","organizations_url":"https://api.github.com/users/thomasdelhomenie/orgs","repos_url":"https://api.github.com/users/thomasdelhomenie/repos","events_url":"https://api.github.com/users/thomasdelhomenie/events{/privacy}","received_events_url":"https://api.github.com/users/thomasdelhomenie/received_events","type":"User","site_admin":false},"body":"Shouldn't we throw an exception in case createdTime is null ? The purppose of this method is to update this variable, so if it cannot be done, the caller should be warned IMO (by getting an exception).","created_at":"2016-11-28T11:28:09Z","updated_at":"2016-11-28T11:28:27Z","html_url":"https://github.com/exodev/ecms/pull/244#discussion_r89762845","pull_request_url":"https://api.github.com/repos/exodev/ecms/pulls/244","_links":{"self":{"href":"https://api.github.com/repos/exodev/ecms/pulls/comments/89762845"},"html":{"href":"https://github.com/exodev/ecms/pull/244#discussion_r89762845"},"pull_request":{"href":"https://api.github.com/repos/exodev/ecms/pulls/244"}}},"pull_request":{"url":"https://api.github.com/repos/exodev/ecms/pulls/244","id":94401194,"html_url":"https://github.com/exodev/ecms/pull/244","diff_url":"https://github.com/exodev/ecms/pull/244.diff","patch_url":"https://github.com/exodev/ecms/pull/244.patch","issue_url":"https://api.github.com/repos/exodev/ecms/issues/244","number":244,"state":"open","locked":false,"title":"ECMS-7516 Error on document uploaded outside collaboration WS","user":{"login":"boubaker","id":594824,"avatar_url":"https://avatars.githubusercontent.com/u/594824?v=3","gravatar_id":"","url":"https://api.github.com/users/boubaker","html_url":"https://github.com/boubaker","followers_url":"https://api.github.com/users/boubaker/followers","following_url":"https://api.github.com/users/boubaker/following{/other_user}","gists_url":"https://api.github.com/users/boubaker/gists{/gist_id}","starred_url":"https://api.github.com/users/boubaker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boubaker/subscriptions","organizations_url":"https://api.github.com/users/boubaker/orgs","repos_url":"https://api.github.com/users/boubaker/repos","events_url":"https://api.github.com/users/boubaker/events{/privacy}","received_events_url":"https://api.github.com/users/boubaker/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-11-18T17:16:45Z","updated_at":"2016-11-28T11:28:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"30dace7dcdce3ac2a54b8321c240b6478e941c13","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/exodev/ecms/pulls/244/commits","review_comments_url":"https://api.github.com/repos/exodev/ecms/pulls/244/comments","review_comment_url":"https://api.github.com/repos/exodev/ecms/pulls/comments{/number}","comments_url":"https://api.github.com/repos/exodev/ecms/issues/244/comments","statuses_url":"https://api.github.com/repos/exodev/ecms/statuses/bbacb43900ebfcc6dd3a64bdff37f90ea2999913","head":{"label":"exodev:fix/ECMS-7516","ref":"fix/ECMS-7516","sha":"bbacb43900ebfcc6dd3a64bdff37f90ea2999913","user":{"login":"exodev","id":1606522,"avatar_url":"https://avatars.githubusercontent.com/u/1606522?v=3","gravatar_id":"","url":"https://api.github.com/users/exodev","html_url":"https://github.com/exodev","followers_url":"https://api.github.com/users/exodev/followers","following_url":"https://api.github.com/users/exodev/following{/other_user}","gists_url":"https://api.github.com/users/exodev/gists{/gist_id}","starred_url":"https://api.github.com/users/exodev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/exodev/subscriptions","organizations_url":"https://api.github.com/users/exodev/orgs","repos_url":"https://api.github.com/users/exodev/repos","events_url":"https://api.github.com/users/exodev/events{/privacy}","received_events_url":"https://api.github.com/users/exodev/received_events","type":"Organization","site_admin":false},"repo":{"id":4003238,"name":"ecms","full_name":"exodev/ecms","owner":{"login":"exodev","id":1606522,"avatar_url":"https://avatars.githubusercontent.com/u/1606522?v=3","gravatar_id":"","url":"https://api.github.com/users/exodev","html_url":"https://github.com/exodev","followers_url":"https://api.github.com/users/exodev/followers","following_url":"https://api.github.com/users/exodev/following{/other_user}","gists_url":"https://api.github.com/users/exodev/gists{/gist_id}","starred_url":"https://api.github.com/users/exodev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/exodev/subscriptions","organizations_url":"https://api.github.com/users/exodev/orgs","repos_url":"https://api.github.com/users/exodev/repos","events_url":"https://api.github.com/users/exodev/events{/privacy}","received_events_url":"https://api.github.com/users/exodev/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/exodev/ecms","description":"eXo ECMS","fork":true,"url":"https://api.github.com/repos/exodev/ecms","forks_url":"https://api.github.com/repos/exodev/ecms/forks","keys_url":"https://api.github.com/repos/exodev/ecms/keys{/key_id}","collaborators_url":"https://api.github.com/repos/exodev/ecms/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/exodev/ecms/teams","hooks_url":"https://api.github.com/repos/exodev/ecms/hooks","issue_events_url":"https://api.github.com/repos/exodev/ecms/issues/events{/number}","events_url":"https://api.github.com/repos/exodev/ecms/events","assignees_url":"https://api.github.com/repos/exodev/ecms/assignees{/user}","branches_url":"https://api.github.com/repos/exodev/ecms/branches{/branch}","tags_url":"https://api.github.com/repos/exodev/ecms/tags","blobs_url":"https://api.github.com/repos/exodev/ecms/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/exodev/ecms/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/exodev/ecms/git/refs{/sha}","trees_url":"https://api.github.com/repos/exodev/ecms/git/trees{/sha}","statuses_url":"https://api.github.com/repos/exodev/ecms/statuses/{sha}","languages_url":"https://api.github.com/repos/exodev/ecms/languages","stargazers_url":"https://api.github.com/repos/exodev/ecms/stargazers","contributors_url":"https://api.github.com/repos/exodev/ecms/contributors","subscribers_url":"https://api.github.com/repos/exodev/ecms/subscribers","subscription_url":"https://api.github.com/repos/exodev/ecms/subscription","commits_url":"https://api.github.com/repos/exodev/ecms/commits{/sha}","git_commits_url":"https://api.github.com/repos/exodev/ecms/git/commits{/sha}","comments_url":"https://api.github.com/repos/exodev/ecms/comments{/number}","issue_comment_url":"https://api.github.com/repos/exodev/ecms/issues/comments{/number}","contents_url":"https://api.github.com/repos/exodev/ecms/contents/{+path}","compare_url":"https://api.github.com/repos/exodev/ecms/compare/{base}...{head}","merges_url":"https://api.github.com/repos/exodev/ecms/merges","archive_url":"https://api.github.com/repos/exodev/ecms/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/exodev/ecms/downloads","issues_url":"https://api.github.com/repos/exodev/ecms/issues{/number}","pulls_url":"https://api.github.com/repos/exodev/ecms/pulls{/number}","milestones_url":"https://api.github.com/repos/exodev/ecms/milestones{/number}","notifications_url":"https://api.github.com/repos/exodev/ecms/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/exodev/ecms/labels{/name}","releases_url":"https://api.github.com/repos/exodev/ecms/releases{/id}","deployments_url":"https://api.github.com/repos/exodev/ecms/deployments","created_at":"2012-04-12T09:40:08Z","updated_at":"2016-08-22T14:21:17Z","pushed_at":"2016-11-27T19:09:51Z","git_url":"git://github.com/exodev/ecms.git","ssh_url":"git@github.com:exodev/ecms.git","clone_url":"https://github.com/exodev/ecms.git","svn_url":"https://github.com/exodev/ecms","homepage":null,"size":188461,"stargazers_count":3,"watchers_count":3,"language":"Java","has_issues":false,"has_downloads":false,"has_wiki":false,"has_pages":false,"forks_count":14,"mirror_url":null,"open_issues_count":95,"forks":14,"open_issues":95,"watchers":3,"default_branch":"develop"}},"base":{"label":"exodev:develop","ref":"develop","sha":"99035cdf5edaa770c5560e6bbed1bf069c007027","user":{"login":"exodev","id":1606522,"avatar_url":"https://avatars.githubusercontent.com/u/1606522?v=3","gravatar_id":"","url":"https://api.github.com/users/exodev","html_url":"https://github.com/exodev","followers_url":"https://api.github.com/users/exodev/followers","following_url":"https://api.github.com/users/exodev/following{/other_user}","gists_url":"https://api.github.com/users/exodev/gists{/gist_id}","starred_url":"https://api.github.com/users/exodev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/exodev/subscriptions","organizations_url":"https://api.github.com/users/exodev/orgs","repos_url":"https://api.github.com/users/exodev/repos","events_url":"https://api.github.com/users/exodev/events{/privacy}","received_events_url":"https://api.github.com/users/exodev/received_events","type":"Organization","site_admin":false},"repo":{"id":4003238,"name":"ecms","full_name":"exodev/ecms","owner":{"login":"exodev","id":1606522,"avatar_url":"https://avatars.githubusercontent.com/u/1606522?v=3","gravatar_id":"","url":"https://api.github.com/users/exodev","html_url":"https://github.com/exodev","followers_url":"https://api.github.com/users/exodev/followers","following_url":"https://api.github.com/users/exodev/following{/other_user}","gists_url":"https://api.github.com/users/exodev/gists{/gist_id}","starred_url":"https://api.github.com/users/exodev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/exodev/subscriptions","organizations_url":"https://api.github.com/users/exodev/orgs","repos_url":"https://api.github.com/users/exodev/repos","events_url":"https://api.github.com/users/exodev/events{/privacy}","received_events_url":"https://api.github.com/users/exodev/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/exodev/ecms","description":"eXo ECMS","fork":true,"url":"https://api.github.com/repos/exodev/ecms","forks_url":"https://api.github.com/repos/exodev/ecms/forks","keys_url":"https://api.github.com/repos/exodev/ecms/keys{/key_id}","collaborators_url":"https://api.github.com/repos/exodev/ecms/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/exodev/ecms/teams","hooks_url":"https://api.github.com/repos/exodev/ecms/hooks","issue_events_url":"https://api.github.com/repos/exodev/ecms/issues/events{/number}","events_url":"https://api.github.com/repos/exodev/ecms/events","assignees_url":"https://api.github.com/repos/exodev/ecms/assignees{/user}","branches_url":"https://api.github.com/repos/exodev/ecms/branches{/branch}","tags_url":"https://api.github.com/repos/exodev/ecms/tags","blobs_url":"https://api.github.com/repos/exodev/ecms/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/exodev/ecms/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/exodev/ecms/git/refs{/sha}","trees_url":"https://api.github.com/repos/exodev/ecms/git/trees{/sha}","statuses_url":"https://api.github.com/repos/exodev/ecms/statuses/{sha}","languages_url":"https://api.github.com/repos/exodev/ecms/languages","stargazers_url":"https://api.github.com/repos/exodev/ecms/stargazers","contributors_url":"https://api.github.com/repos/exodev/ecms/contributors","subscribers_url":"https://api.github.com/repos/exodev/ecms/subscribers","subscription_url":"https://api.github.com/repos/exodev/ecms/subscription","commits_url":"https://api.github.com/repos/exodev/ecms/commits{/sha}","git_commits_url":"https://api.github.com/repos/exodev/ecms/git/commits{/sha}","comments_url":"https://api.github.com/repos/exodev/ecms/comments{/number}","issue_comment_url":"https://api.github.com/repos/exodev/ecms/issues/comments{/number}","contents_url":"https://api.github.com/repos/exodev/ecms/contents/{+path}","compare_url":"https://api.github.com/repos/exodev/ecms/compare/{base}...{head}","merges_url":"https://api.github.com/repos/exodev/ecms/merges","archive_url":"https://api.github.com/repos/exodev/ecms/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/exodev/ecms/downloads","issues_url":"https://api.github.com/repos/exodev/ecms/issues{/number}","pulls_url":"https://api.github.com/repos/exodev/ecms/pulls{/number}","milestones_url":"https://api.github.com/repos/exodev/ecms/milestones{/number}","notifications_url":"https://api.github.com/repos/exodev/ecms/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/exodev/ecms/labels{/name}","releases_url":"https://api.github.com/repos/exodev/ecms/releases{/id}","deployments_url":"https://api.github.com/repos/exodev/ecms/deployments","created_at":"2012-04-12T09:40:08Z","updated_at":"2016-08-22T14:21:17Z","pushed_at":"2016-11-27T19:09:51Z","git_url":"git://github.com/exodev/ecms.git","ssh_url":"git@github.com:exodev/ecms.git","clone_url":"https://github.com/exodev/ecms.git","svn_url":"https://github.com/exodev/ecms","homepage":null,"size":188461,"stargazers_count":3,"watchers_count":3,"language":"Java","has_issues":false,"has_downloads":false,"has_wiki":false,"has_pages":false,"forks_count":14,"mirror_url":null,"open_issues_count":95,"forks":14,"open_issues":95,"watchers":3,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/exodev/ecms/pulls/244"},"html":{"href":"https://github.com/exodev/ecms/pull/244"},"issue":{"href":"https://api.github.com/repos/exodev/ecms/issues/244"},"comments":{"href":"https://api.github.com/repos/exodev/ecms/issues/244/comments"},"review_comments":{"href":"https://api.github.com/repos/exodev/ecms/pulls/244/comments"},"review_comment":{"href":"https://api.github.com/repos/exodev/ecms/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/exodev/ecms/pulls/244/commits"},"statuses":{"href":"https://api.github.com/repos/exodev/ecms/statuses/bbacb43900ebfcc6dd3a64bdff37f90ea2999913"}}}}
{"actor":{"display_login":"thomasdelhomenie"}}