Consistent name with other classes

kohsuke · kohsuke · commit e239ef50bac1 · 2015-07-17T14:06:35.000+03:00
diff --git a/src/main/java/org/kohsuke/github/GHHook.java b/src/main/java/org/kohsuke/github/GHHook.java
@@ -40,7 +40,7 @@ public Map<String, String> getConfig() {
      * Deletes this hook.
      */
     public void delete() throws IOException {
-        new Requester(root()).method("DELETE").to(path());
+        new Requester(getRoot()).method("DELETE").to(getApiRoute());
     }
 
     /**
@@ -51,7 +51,7 @@ public URL getHtmlUrl() {
         return null;
     }
 
-    abstract GitHub root();
+    abstract GitHub getRoot();
 
-    abstract String path();
+    abstract String getApiRoute();
 }
diff --git a/src/main/java/org/kohsuke/github/GHOrgHook.java b/src/main/java/org/kohsuke/github/GHOrgHook.java
@@ -16,12 +16,12 @@ class GHOrgHook extends GHHook {
     }
 
     @Override
-    GitHub root() {
+    GitHub getRoot() {
         return organization.root;
     }
 
     @Override
-    String path() {
+    String getApiRoute() {
         return String.format("/orgs/%s/hooks/%d", organization.getLogin(), id);
     }
 }
diff --git a/src/main/java/org/kohsuke/github/GHRepoHook.java b/src/main/java/org/kohsuke/github/GHRepoHook.java
@@ -12,12 +12,12 @@ class GHRepoHook extends GHHook {
     }
 
     @Override
-    GitHub root() {
+    GitHub getRoot() {
         return repository.root;
     }
 
     @Override
-    String path() {
+    String getApiRoute() {
         return String.format("/repos/%s/%s/hooks/%d", repository.getOwnerName(), repository.getName(), id);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ public Map<String, String> getConfig() {`
`40`	`40`	`* Deletes this hook.`
`41`	`41`	`*/`
`42`	`42`	`public void delete() throws IOException {`
`43`		`- new Requester(root()).method("DELETE").to(path());`
	`43`	`+ new Requester(getRoot()).method("DELETE").to(getApiRoute());`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`/**`
`@@ -51,7 +51,7 @@ public URL getHtmlUrl() {`
`51`	`51`	`return null;`
`52`	`52`	`}`
`53`	`53`
`54`		`- abstract GitHub root();`
	`54`	`+ abstract GitHub getRoot();`
`55`	`55`
`56`		`- abstract String path();`
	`56`	`+ abstract String getApiRoute();`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,12 +16,12 @@ class GHOrgHook extends GHHook {`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`@Override`
`19`		`- GitHub root() {`
	`19`	`+ GitHub getRoot() {`
`20`	`20`	`return organization.root;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`@Override`
`24`		`- String path() {`
	`24`	`+ String getApiRoute() {`
`25`	`25`	`return String.format("/orgs/%s/hooks/%d", organization.getLogin(), id);`
`26`	`26`	`}`
`27`	`27`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,12 +12,12 @@ class GHRepoHook extends GHHook {`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`@Override`
`15`		`- GitHub root() {`
	`15`	`+ GitHub getRoot() {`
`16`	`16`	`return repository.root;`
`17`	`17`	`}`
`18`	`18`
`19`	`19`	`@Override`
`20`		`- String path() {`
	`20`	`+ String getApiRoute() {`
`21`	`21`	`return String.format("/repos/%s/%s/hooks/%d", repository.getOwnerName(), repository.getName(), id);`
`22`	`22`	`}`
`23`	`23`	`}`