{"action":"created","comment":{"url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/comments/40407772","id":40407772,"diff_hunk":"@@ -176,6 +181,18 @@ public ChangeMatrixOpImage(final RenderedImage reference, final RenderedImage no\n         // Area grid is present\n         areaGrid = area != null;\n         areaMap = area;\n+        \n+        // is NoData range present?\n+        this.hasNoData = (noData!=null)?true:false;\n+        this.noData = noData;\n+        if(this.hasNoData){\n+            // Check if No ChangeMatrix classes set belong to NoData\n+            for(Integer clazz : result.getRegisteredClasses()){\n+                if(noData.contains(clazz)){\n+                    throw new IllegalStateException(\"One or more provided classes are contained in the NoData range\");\n+                }\n+            }\n+        }\n     }","path":"changematrix/src/main/java/it/geosolutions/jaiext/changematrix/ChangeMatrixOpImage.java","position":64,"original_position":64,"commit_id":"e8ce597481d050708070ee870e1d3e33e746466c","original_commit_id":"e8ce597481d050708070ee870e1d3e33e746466c","user":{"login":"dromagnoli","id":1278116,"avatar_url":"https://avatars.githubusercontent.com/u/1278116?v=3","gravatar_id":"","url":"https://api.github.com/users/dromagnoli","html_url":"https://github.com/dromagnoli","followers_url":"https://api.github.com/users/dromagnoli/followers","following_url":"https://api.github.com/users/dromagnoli/following{/other_user}","gists_url":"https://api.github.com/users/dromagnoli/gists{/gist_id}","starred_url":"https://api.github.com/users/dromagnoli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dromagnoli/subscriptions","organizations_url":"https://api.github.com/users/dromagnoli/orgs","repos_url":"https://api.github.com/users/dromagnoli/repos","events_url":"https://api.github.com/users/dromagnoli/events{/privacy}","received_events_url":"https://api.github.com/users/dromagnoli/received_events","type":"User","site_admin":false},"body":"For new lines of code, make sure to use proper formatting templates.\r\nif(condition){ \r\n\r\nshould be\r\n\r\nif (condition) {\r\n\r\nsame for for and so on.\r\n","created_at":"2015-09-25T08:08:32Z","updated_at":"2015-09-25T08:08:32Z","html_url":"https://github.com/geosolutions-it/jai-ext/pull/90#discussion_r40407772","pull_request_url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90","_links":{"self":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/comments/40407772"},"html":{"href":"https://github.com/geosolutions-it/jai-ext/pull/90#discussion_r40407772"},"pull_request":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90"}}},"pull_request":{"url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90","id":45199448,"html_url":"https://github.com/geosolutions-it/jai-ext/pull/90","diff_url":"https://github.com/geosolutions-it/jai-ext/pull/90.diff","patch_url":"https://github.com/geosolutions-it/jai-ext/pull/90.patch","issue_url":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/90","number":90,"state":"open","locked":false,"title":"close issue #89 and added referenceArea tests","user":{"login":"Damianofds","id":1670618,"avatar_url":"https://avatars.githubusercontent.com/u/1670618?v=3","gravatar_id":"","url":"https://api.github.com/users/Damianofds","html_url":"https://github.com/Damianofds","followers_url":"https://api.github.com/users/Damianofds/followers","following_url":"https://api.github.com/users/Damianofds/following{/other_user}","gists_url":"https://api.github.com/users/Damianofds/gists{/gist_id}","starred_url":"https://api.github.com/users/Damianofds/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Damianofds/subscriptions","organizations_url":"https://api.github.com/users/Damianofds/orgs","repos_url":"https://api.github.com/users/Damianofds/repos","events_url":"https://api.github.com/users/Damianofds/events{/privacy}","received_events_url":"https://api.github.com/users/Damianofds/received_events","type":"User","site_admin":false},"body":"","created_at":"2015-09-16T17:19:04Z","updated_at":"2015-09-25T08:08:32Z","closed_at":null,"merged_at":null,"merge_commit_sha":"40f260ee5ee74f793d30b377dd088c19886eb94e","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90/commits","review_comments_url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90/comments","review_comment_url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/comments{/number}","comments_url":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/90/comments","statuses_url":"https://api.github.com/repos/geosolutions-it/jai-ext/statuses/e8ce597481d050708070ee870e1d3e33e746466c","head":{"label":"Damianofds:legacy","ref":"legacy","sha":"e8ce597481d050708070ee870e1d3e33e746466c","user":{"login":"Damianofds","id":1670618,"avatar_url":"https://avatars.githubusercontent.com/u/1670618?v=3","gravatar_id":"","url":"https://api.github.com/users/Damianofds","html_url":"https://github.com/Damianofds","followers_url":"https://api.github.com/users/Damianofds/followers","following_url":"https://api.github.com/users/Damianofds/following{/other_user}","gists_url":"https://api.github.com/users/Damianofds/gists{/gist_id}","starred_url":"https://api.github.com/users/Damianofds/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Damianofds/subscriptions","organizations_url":"https://api.github.com/users/Damianofds/orgs","repos_url":"https://api.github.com/users/Damianofds/repos","events_url":"https://api.github.com/users/Damianofds/events{/privacy}","received_events_url":"https://api.github.com/users/Damianofds/received_events","type":"User","site_admin":false},"repo":{"id":42295792,"name":"jai-ext","full_name":"Damianofds/jai-ext","owner":{"login":"Damianofds","id":1670618,"avatar_url":"https://avatars.githubusercontent.com/u/1670618?v=3","gravatar_id":"","url":"https://api.github.com/users/Damianofds","html_url":"https://github.com/Damianofds","followers_url":"https://api.github.com/users/Damianofds/followers","following_url":"https://api.github.com/users/Damianofds/following{/other_user}","gists_url":"https://api.github.com/users/Damianofds/gists{/gist_id}","starred_url":"https://api.github.com/users/Damianofds/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Damianofds/subscriptions","organizations_url":"https://api.github.com/users/Damianofds/orgs","repos_url":"https://api.github.com/users/Damianofds/repos","events_url":"https://api.github.com/users/Damianofds/events{/privacy}","received_events_url":"https://api.github.com/users/Damianofds/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/Damianofds/jai-ext","description":"Oracle JAI Open Source Replacement Wannabe","fork":true,"url":"https://api.github.com/repos/Damianofds/jai-ext","forks_url":"https://api.github.com/repos/Damianofds/jai-ext/forks","keys_url":"https://api.github.com/repos/Damianofds/jai-ext/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Damianofds/jai-ext/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Damianofds/jai-ext/teams","hooks_url":"https://api.github.com/repos/Damianofds/jai-ext/hooks","issue_events_url":"https://api.github.com/repos/Damianofds/jai-ext/issues/events{/number}","events_url":"https://api.github.com/repos/Damianofds/jai-ext/events","assignees_url":"https://api.github.com/repos/Damianofds/jai-ext/assignees{/user}","branches_url":"https://api.github.com/repos/Damianofds/jai-ext/branches{/branch}","tags_url":"https://api.github.com/repos/Damianofds/jai-ext/tags","blobs_url":"https://api.github.com/repos/Damianofds/jai-ext/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Damianofds/jai-ext/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Damianofds/jai-ext/git/refs{/sha}","trees_url":"https://api.github.com/repos/Damianofds/jai-ext/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Damianofds/jai-ext/statuses/{sha}","languages_url":"https://api.github.com/repos/Damianofds/jai-ext/languages","stargazers_url":"https://api.github.com/repos/Damianofds/jai-ext/stargazers","contributors_url":"https://api.github.com/repos/Damianofds/jai-ext/contributors","subscribers_url":"https://api.github.com/repos/Damianofds/jai-ext/subscribers","subscription_url":"https://api.github.com/repos/Damianofds/jai-ext/subscription","commits_url":"https://api.github.com/repos/Damianofds/jai-ext/commits{/sha}","git_commits_url":"https://api.github.com/repos/Damianofds/jai-ext/git/commits{/sha}","comments_url":"https://api.github.com/repos/Damianofds/jai-ext/comments{/number}","issue_comment_url":"https://api.github.com/repos/Damianofds/jai-ext/issues/comments{/number}","contents_url":"https://api.github.com/repos/Damianofds/jai-ext/contents/{+path}","compare_url":"https://api.github.com/repos/Damianofds/jai-ext/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Damianofds/jai-ext/merges","archive_url":"https://api.github.com/repos/Damianofds/jai-ext/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Damianofds/jai-ext/downloads","issues_url":"https://api.github.com/repos/Damianofds/jai-ext/issues{/number}","pulls_url":"https://api.github.com/repos/Damianofds/jai-ext/pulls{/number}","milestones_url":"https://api.github.com/repos/Damianofds/jai-ext/milestones{/number}","notifications_url":"https://api.github.com/repos/Damianofds/jai-ext/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Damianofds/jai-ext/labels{/name}","releases_url":"https://api.github.com/repos/Damianofds/jai-ext/releases{/id}","created_at":"2015-09-11T08:03:54Z","updated_at":"2015-09-11T08:03:57Z","pushed_at":"2015-09-16T17:14:26Z","git_url":"git://github.com/Damianofds/jai-ext.git","ssh_url":"git@github.com:Damianofds/jai-ext.git","clone_url":"https://github.com/Damianofds/jai-ext.git","svn_url":"https://github.com/Damianofds/jai-ext","homepage":"","size":22676,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"geosolutions-it:legacy","ref":"legacy","sha":"a82cdd267928b152e712d892dd0e37466e9fc86f","user":{"login":"geosolutions-it","id":775710,"avatar_url":"https://avatars.githubusercontent.com/u/775710?v=3","gravatar_id":"","url":"https://api.github.com/users/geosolutions-it","html_url":"https://github.com/geosolutions-it","followers_url":"https://api.github.com/users/geosolutions-it/followers","following_url":"https://api.github.com/users/geosolutions-it/following{/other_user}","gists_url":"https://api.github.com/users/geosolutions-it/gists{/gist_id}","starred_url":"https://api.github.com/users/geosolutions-it/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geosolutions-it/subscriptions","organizations_url":"https://api.github.com/users/geosolutions-it/orgs","repos_url":"https://api.github.com/users/geosolutions-it/repos","events_url":"https://api.github.com/users/geosolutions-it/events{/privacy}","received_events_url":"https://api.github.com/users/geosolutions-it/received_events","type":"Organization","site_admin":false},"repo":{"id":5947247,"name":"jai-ext","full_name":"geosolutions-it/jai-ext","owner":{"login":"geosolutions-it","id":775710,"avatar_url":"https://avatars.githubusercontent.com/u/775710?v=3","gravatar_id":"","url":"https://api.github.com/users/geosolutions-it","html_url":"https://github.com/geosolutions-it","followers_url":"https://api.github.com/users/geosolutions-it/followers","following_url":"https://api.github.com/users/geosolutions-it/following{/other_user}","gists_url":"https://api.github.com/users/geosolutions-it/gists{/gist_id}","starred_url":"https://api.github.com/users/geosolutions-it/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geosolutions-it/subscriptions","organizations_url":"https://api.github.com/users/geosolutions-it/orgs","repos_url":"https://api.github.com/users/geosolutions-it/repos","events_url":"https://api.github.com/users/geosolutions-it/events{/privacy}","received_events_url":"https://api.github.com/users/geosolutions-it/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/geosolutions-it/jai-ext","description":"Oracle JAI Open Source Replacement Wannabe","fork":false,"url":"https://api.github.com/repos/geosolutions-it/jai-ext","forks_url":"https://api.github.com/repos/geosolutions-it/jai-ext/forks","keys_url":"https://api.github.com/repos/geosolutions-it/jai-ext/keys{/key_id}","collaborators_url":"https://api.github.com/repos/geosolutions-it/jai-ext/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/geosolutions-it/jai-ext/teams","hooks_url":"https://api.github.com/repos/geosolutions-it/jai-ext/hooks","issue_events_url":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/events{/number}","events_url":"https://api.github.com/repos/geosolutions-it/jai-ext/events","assignees_url":"https://api.github.com/repos/geosolutions-it/jai-ext/assignees{/user}","branches_url":"https://api.github.com/repos/geosolutions-it/jai-ext/branches{/branch}","tags_url":"https://api.github.com/repos/geosolutions-it/jai-ext/tags","blobs_url":"https://api.github.com/repos/geosolutions-it/jai-ext/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/geosolutions-it/jai-ext/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/geosolutions-it/jai-ext/git/refs{/sha}","trees_url":"https://api.github.com/repos/geosolutions-it/jai-ext/git/trees{/sha}","statuses_url":"https://api.github.com/repos/geosolutions-it/jai-ext/statuses/{sha}","languages_url":"https://api.github.com/repos/geosolutions-it/jai-ext/languages","stargazers_url":"https://api.github.com/repos/geosolutions-it/jai-ext/stargazers","contributors_url":"https://api.github.com/repos/geosolutions-it/jai-ext/contributors","subscribers_url":"https://api.github.com/repos/geosolutions-it/jai-ext/subscribers","subscription_url":"https://api.github.com/repos/geosolutions-it/jai-ext/subscription","commits_url":"https://api.github.com/repos/geosolutions-it/jai-ext/commits{/sha}","git_commits_url":"https://api.github.com/repos/geosolutions-it/jai-ext/git/commits{/sha}","comments_url":"https://api.github.com/repos/geosolutions-it/jai-ext/comments{/number}","issue_comment_url":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/comments{/number}","contents_url":"https://api.github.com/repos/geosolutions-it/jai-ext/contents/{+path}","compare_url":"https://api.github.com/repos/geosolutions-it/jai-ext/compare/{base}...{head}","merges_url":"https://api.github.com/repos/geosolutions-it/jai-ext/merges","archive_url":"https://api.github.com/repos/geosolutions-it/jai-ext/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/geosolutions-it/jai-ext/downloads","issues_url":"https://api.github.com/repos/geosolutions-it/jai-ext/issues{/number}","pulls_url":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls{/number}","milestones_url":"https://api.github.com/repos/geosolutions-it/jai-ext/milestones{/number}","notifications_url":"https://api.github.com/repos/geosolutions-it/jai-ext/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/geosolutions-it/jai-ext/labels{/name}","releases_url":"https://api.github.com/repos/geosolutions-it/jai-ext/releases{/id}","created_at":"2012-09-25T08:47:49Z","updated_at":"2015-09-11T16:06:52Z","pushed_at":"2015-09-16T17:19:04Z","git_url":"git://github.com/geosolutions-it/jai-ext.git","ssh_url":"git@github.com:geosolutions-it/jai-ext.git","clone_url":"https://github.com/geosolutions-it/jai-ext.git","svn_url":"https://github.com/geosolutions-it/jai-ext","homepage":"","size":23498,"stargazers_count":14,"watchers_count":14,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":10,"mirror_url":null,"open_issues_count":18,"forks":10,"open_issues":18,"watchers":14,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90"},"html":{"href":"https://github.com/geosolutions-it/jai-ext/pull/90"},"issue":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/90"},"comments":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/issues/90/comments"},"review_comments":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90/comments"},"review_comment":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/pulls/90/commits"},"statuses":{"href":"https://api.github.com/repos/geosolutions-it/jai-ext/statuses/e8ce597481d050708070ee870e1d3e33e746466c"}}}}