{"action":"created","comment":{"url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/comments/207025076","pull_request_review_id":142555391,"id":207025076,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwNzAyNTA3Ng==","diff_hunk":"@@ -17,29 +17,35 @@ public class WorkspaceLayoutPersistor : IWorkspaceLayoutPersistor\n       private readonly IWorkspaceLayoutMetaDataToWorkspaceLayoutMapper _workspaceLayoutMapper;\n \n       public WorkspaceLayoutPersistor(IWorkspaceLayoutToWorkspaceLayoutMetaDataMapper workspaceLayoutMetaDataMapper,\n-                                      IWorkspaceLayoutMetaDataToWorkspaceLayoutMapper workspaceLayoutMapper)\n+         IWorkspaceLayoutMetaDataToWorkspaceLayoutMapper workspaceLayoutMapper)\n       {\n          _workspaceLayoutMetaDataMapper = workspaceLayoutMetaDataMapper;\n          _workspaceLayoutMapper = workspaceLayoutMapper;\n       }\n \n       public void Save(IWorkspaceLayout workspaceLayout, ISession session)\n       {\n-\n-         //first remove old layout if existing\n-         var layoutsFromDb = session.CreateCriteria<WorkspaceLayoutMetaData>().List<WorkspaceLayoutMetaData>();\n-         layoutsFromDb.Each(session.Delete);\n+         var layoutFromDb = loadLayoutFromDb(session);","path":"src/PKSim.Infrastructure/Serialization/WorkspaceLayoutPersistor.cs","position":30,"original_position":30,"commit_id":"e56c993c5281e63566e9feeaa4f6761f5919ce67","original_commit_id":"e56c993c5281e63566e9feeaa4f6761f5919ce67","user":{"login":"msevestre","id":1041237,"node_id":"MDQ6VXNlcjEwNDEyMzc=","avatar_url":"https://avatars2.githubusercontent.com/u/1041237?v=4","gravatar_id":"","url":"https://api.github.com/users/msevestre","html_url":"https://github.com/msevestre","followers_url":"https://api.github.com/users/msevestre/followers","following_url":"https://api.github.com/users/msevestre/following{/other_user}","gists_url":"https://api.github.com/users/msevestre/gists{/gist_id}","starred_url":"https://api.github.com/users/msevestre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msevestre/subscriptions","organizations_url":"https://api.github.com/users/msevestre/orgs","repos_url":"https://api.github.com/users/msevestre/repos","events_url":"https://api.github.com/users/msevestre/events{/privacy}","received_events_url":"https://api.github.com/users/msevestre/received_events","type":"User","site_admin":false},"body":"I realized that the content was always deleted and added. A little stupid and also annoying when looking at what is changing when saving a project. That way, we update the existing layout if there is already one instead of saving it again ","created_at":"2018-08-01T20:44:29Z","updated_at":"2018-08-01T20:44:39Z","html_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905#discussion_r207025076","pull_request_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/comments/207025076"},"html":{"href":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905#discussion_r207025076"},"pull_request":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905"}}},"pull_request":{"url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905","id":205547535,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA1NTQ3NTM1","html_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905","diff_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905.diff","patch_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905.patch","issue_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/905","number":905,"state":"open","locked":false,"title":"Fixes #904 content data not deleted","user":{"login":"msevestre","id":1041237,"node_id":"MDQ6VXNlcjEwNDEyMzc=","avatar_url":"https://avatars2.githubusercontent.com/u/1041237?v=4","gravatar_id":"","url":"https://api.github.com/users/msevestre","html_url":"https://github.com/msevestre","followers_url":"https://api.github.com/users/msevestre/followers","following_url":"https://api.github.com/users/msevestre/following{/other_user}","gists_url":"https://api.github.com/users/msevestre/gists{/gist_id}","starred_url":"https://api.github.com/users/msevestre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msevestre/subscriptions","organizations_url":"https://api.github.com/users/msevestre/orgs","repos_url":"https://api.github.com/users/msevestre/repos","events_url":"https://api.github.com/users/msevestre/events{/privacy}","received_events_url":"https://api.github.com/users/msevestre/received_events","type":"User","site_admin":false},"body":"","created_at":"2018-08-01T20:43:29Z","updated_at":"2018-08-01T20:44:39Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5ee61bfd1782459dd228af5c0e33168520e0c02e","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905/commits","review_comments_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905/comments","review_comment_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/905/comments","statuses_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/statuses/e56c993c5281e63566e9feeaa4f6761f5919ce67","head":{"label":"msevestre:904-content-data-not-deleted","ref":"904-content-data-not-deleted","sha":"e56c993c5281e63566e9feeaa4f6761f5919ce67","user":{"login":"msevestre","id":1041237,"node_id":"MDQ6VXNlcjEwNDEyMzc=","avatar_url":"https://avatars2.githubusercontent.com/u/1041237?v=4","gravatar_id":"","url":"https://api.github.com/users/msevestre","html_url":"https://github.com/msevestre","followers_url":"https://api.github.com/users/msevestre/followers","following_url":"https://api.github.com/users/msevestre/following{/other_user}","gists_url":"https://api.github.com/users/msevestre/gists{/gist_id}","starred_url":"https://api.github.com/users/msevestre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msevestre/subscriptions","organizations_url":"https://api.github.com/users/msevestre/orgs","repos_url":"https://api.github.com/users/msevestre/repos","events_url":"https://api.github.com/users/msevestre/events{/privacy}","received_events_url":"https://api.github.com/users/msevestre/received_events","type":"User","site_admin":false},"repo":{"id":86487080,"node_id":"MDEwOlJlcG9zaXRvcnk4NjQ4NzA4MA==","name":"PK-Sim","full_name":"msevestre/PK-Sim","owner":{"login":"msevestre","id":1041237,"node_id":"MDQ6VXNlcjEwNDEyMzc=","avatar_url":"https://avatars2.githubusercontent.com/u/1041237?v=4","gravatar_id":"","url":"https://api.github.com/users/msevestre","html_url":"https://github.com/msevestre","followers_url":"https://api.github.com/users/msevestre/followers","following_url":"https://api.github.com/users/msevestre/following{/other_user}","gists_url":"https://api.github.com/users/msevestre/gists{/gist_id}","starred_url":"https://api.github.com/users/msevestre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msevestre/subscriptions","organizations_url":"https://api.github.com/users/msevestre/orgs","repos_url":"https://api.github.com/users/msevestre/repos","events_url":"https://api.github.com/users/msevestre/events{/privacy}","received_events_url":"https://api.github.com/users/msevestre/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/msevestre/PK-Sim","description":"PK-Sim® is a comprehensive software tool for whole-body physiologically based pharmacokinetic modeling","fork":true,"url":"https://api.github.com/repos/msevestre/PK-Sim","forks_url":"https://api.github.com/repos/msevestre/PK-Sim/forks","keys_url":"https://api.github.com/repos/msevestre/PK-Sim/keys{/key_id}","collaborators_url":"https://api.github.com/repos/msevestre/PK-Sim/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/msevestre/PK-Sim/teams","hooks_url":"https://api.github.com/repos/msevestre/PK-Sim/hooks","issue_events_url":"https://api.github.com/repos/msevestre/PK-Sim/issues/events{/number}","events_url":"https://api.github.com/repos/msevestre/PK-Sim/events","assignees_url":"https://api.github.com/repos/msevestre/PK-Sim/assignees{/user}","branches_url":"https://api.github.com/repos/msevestre/PK-Sim/branches{/branch}","tags_url":"https://api.github.com/repos/msevestre/PK-Sim/tags","blobs_url":"https://api.github.com/repos/msevestre/PK-Sim/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/msevestre/PK-Sim/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/msevestre/PK-Sim/git/refs{/sha}","trees_url":"https://api.github.com/repos/msevestre/PK-Sim/git/trees{/sha}","statuses_url":"https://api.github.com/repos/msevestre/PK-Sim/statuses/{sha}","languages_url":"https://api.github.com/repos/msevestre/PK-Sim/languages","stargazers_url":"https://api.github.com/repos/msevestre/PK-Sim/stargazers","contributors_url":"https://api.github.com/repos/msevestre/PK-Sim/contributors","subscribers_url":"https://api.github.com/repos/msevestre/PK-Sim/subscribers","subscription_url":"https://api.github.com/repos/msevestre/PK-Sim/subscription","commits_url":"https://api.github.com/repos/msevestre/PK-Sim/commits{/sha}","git_commits_url":"https://api.github.com/repos/msevestre/PK-Sim/git/commits{/sha}","comments_url":"https://api.github.com/repos/msevestre/PK-Sim/comments{/number}","issue_comment_url":"https://api.github.com/repos/msevestre/PK-Sim/issues/comments{/number}","contents_url":"https://api.github.com/repos/msevestre/PK-Sim/contents/{+path}","compare_url":"https://api.github.com/repos/msevestre/PK-Sim/compare/{base}...{head}","merges_url":"https://api.github.com/repos/msevestre/PK-Sim/merges","archive_url":"https://api.github.com/repos/msevestre/PK-Sim/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/msevestre/PK-Sim/downloads","issues_url":"https://api.github.com/repos/msevestre/PK-Sim/issues{/number}","pulls_url":"https://api.github.com/repos/msevestre/PK-Sim/pulls{/number}","milestones_url":"https://api.github.com/repos/msevestre/PK-Sim/milestones{/number}","notifications_url":"https://api.github.com/repos/msevestre/PK-Sim/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/msevestre/PK-Sim/labels{/name}","releases_url":"https://api.github.com/repos/msevestre/PK-Sim/releases{/id}","deployments_url":"https://api.github.com/repos/msevestre/PK-Sim/deployments","created_at":"2017-03-28T17:17:08Z","updated_at":"2017-03-28T17:17:16Z","pushed_at":"2018-08-01T20:43:18Z","git_url":"git://github.com/msevestre/PK-Sim.git","ssh_url":"git@github.com:msevestre/PK-Sim.git","clone_url":"https://github.com/msevestre/PK-Sim.git","svn_url":"https://github.com/msevestre/PK-Sim","homepage":"","size":187570,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":null,"url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"develop"}},"base":{"label":"Open-Systems-Pharmacology:develop","ref":"develop","sha":"3d9577f9da2c72a2e7173139c1fd14cdd8bd1d76","user":{"login":"Open-Systems-Pharmacology","id":25098323,"node_id":"MDEyOk9yZ2FuaXphdGlvbjI1MDk4MzIz","avatar_url":"https://avatars1.githubusercontent.com/u/25098323?v=4","gravatar_id":"","url":"https://api.github.com/users/Open-Systems-Pharmacology","html_url":"https://github.com/Open-Systems-Pharmacology","followers_url":"https://api.github.com/users/Open-Systems-Pharmacology/followers","following_url":"https://api.github.com/users/Open-Systems-Pharmacology/following{/other_user}","gists_url":"https://api.github.com/users/Open-Systems-Pharmacology/gists{/gist_id}","starred_url":"https://api.github.com/users/Open-Systems-Pharmacology/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Open-Systems-Pharmacology/subscriptions","organizations_url":"https://api.github.com/users/Open-Systems-Pharmacology/orgs","repos_url":"https://api.github.com/users/Open-Systems-Pharmacology/repos","events_url":"https://api.github.com/users/Open-Systems-Pharmacology/events{/privacy}","received_events_url":"https://api.github.com/users/Open-Systems-Pharmacology/received_events","type":"Organization","site_admin":false},"repo":{"id":81575614,"node_id":"MDEwOlJlcG9zaXRvcnk4MTU3NTYxNA==","name":"PK-Sim","full_name":"Open-Systems-Pharmacology/PK-Sim","owner":{"login":"Open-Systems-Pharmacology","id":25098323,"node_id":"MDEyOk9yZ2FuaXphdGlvbjI1MDk4MzIz","avatar_url":"https://avatars1.githubusercontent.com/u/25098323?v=4","gravatar_id":"","url":"https://api.github.com/users/Open-Systems-Pharmacology","html_url":"https://github.com/Open-Systems-Pharmacology","followers_url":"https://api.github.com/users/Open-Systems-Pharmacology/followers","following_url":"https://api.github.com/users/Open-Systems-Pharmacology/following{/other_user}","gists_url":"https://api.github.com/users/Open-Systems-Pharmacology/gists{/gist_id}","starred_url":"https://api.github.com/users/Open-Systems-Pharmacology/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Open-Systems-Pharmacology/subscriptions","organizations_url":"https://api.github.com/users/Open-Systems-Pharmacology/orgs","repos_url":"https://api.github.com/users/Open-Systems-Pharmacology/repos","events_url":"https://api.github.com/users/Open-Systems-Pharmacology/events{/privacy}","received_events_url":"https://api.github.com/users/Open-Systems-Pharmacology/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim","description":"PK-Sim® is a comprehensive software tool for whole-body physiologically based pharmacokinetic modeling","fork":false,"url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim","forks_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/forks","keys_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/teams","hooks_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/hooks","issue_events_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/events{/number}","events_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/events","assignees_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/assignees{/user}","branches_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/branches{/branch}","tags_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/tags","blobs_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/git/refs{/sha}","trees_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/statuses/{sha}","languages_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/languages","stargazers_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/stargazers","contributors_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/contributors","subscribers_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/subscribers","subscription_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/subscription","commits_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/commits{/sha}","git_commits_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/git/commits{/sha}","comments_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/comments{/number}","issue_comment_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/comments{/number}","contents_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/contents/{+path}","compare_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/merges","archive_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/downloads","issues_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues{/number}","pulls_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls{/number}","milestones_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/milestones{/number}","notifications_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/labels{/name}","releases_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/releases{/id}","deployments_url":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/deployments","created_at":"2017-02-10T14:58:51Z","updated_at":"2018-07-30T15:05:06Z","pushed_at":"2018-08-01T20:43:29Z","git_url":"git://github.com/Open-Systems-Pharmacology/PK-Sim.git","ssh_url":"git@github.com:Open-Systems-Pharmacology/PK-Sim.git","clone_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim.git","svn_url":"https://github.com/Open-Systems-Pharmacology/PK-Sim","homepage":"","size":187692,"stargazers_count":18,"watchers_count":18,"language":"C#","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":9,"mirror_url":null,"archived":false,"open_issues_count":160,"license":{"key":"other","name":"Other","spdx_id":null,"url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":9,"open_issues":160,"watchers":18,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905"},"html":{"href":"https://github.com/Open-Systems-Pharmacology/PK-Sim/pull/905"},"issue":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/905"},"comments":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/issues/905/comments"},"review_comments":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905/comments"},"review_comment":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/pulls/905/commits"},"statuses":{"href":"https://api.github.com/repos/Open-Systems-Pharmacology/PK-Sim/statuses/e56c993c5281e63566e9feeaa4f6761f5919ce67"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"msevestre"}}