Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Replace underscored variable names with camel case (oops)

  • Loading branch information...
commit d33e9ad30eeea6a17fcac9421afc92281d4a194a 1 parent 49d0960
@thickpaddy thickpaddy authored
Showing with 15 additions and 15 deletions.
  1. +15 −15 src/main/java/hudson/plugins/campfire/CampfireNotifier.java
View
30 src/main/java/hudson/plugins/campfire/CampfireNotifier.java
@@ -67,7 +67,7 @@ private void publish(AbstractBuild<?, ?> build) throws IOException {
ChangeLogSet.Entry entry = build.getChangeSet().iterator().next();
// note: iterator should return recent changes first, but GitChangeSetList currently reverses the log entries
if (changeSet.getClass().getSimpleName().equals("GitChangeSetList")) {
- String exception_log_message = "Workaround to obtain latest commit info from git plugin failed";
+ String exceptionLogMsg = "Workaround to obtain latest commit info from git plugin failed";
try {
// find the sha for the first commit in the changelog file, and then grab the corresponding entry from the changeset, yikes!
String changeLogPath = build.getRootDir().toString() + File.separator + "changelog.xml";
@@ -88,13 +88,13 @@ private void publish(AbstractBuild<?, ?> build) throws IOException {
}
}
} catch ( IOException e ){
- LOGGER.log(Level.WARNING, exception_log_message, e);
+ LOGGER.log(Level.WARNING, exceptionLogMsg, e);
} catch ( NoSuchMethodException e ) {
- LOGGER.log(Level.WARNING, exception_log_message, e);
+ LOGGER.log(Level.WARNING, exceptionLogMsg, e);
} catch ( IllegalAccessException e ) {
- LOGGER.log(Level.WARNING, exception_log_message, e);
+ LOGGER.log(Level.WARNING, exceptionLogMsg, e);
} catch ( SecurityException e ) {
- LOGGER.log(Level.WARNING, exception_log_message, e);
+ LOGGER.log(Level.WARNING, exceptionLogMsg, e);
} catch ( Exception e ) {
throw new RuntimeException(e.getClass().getName() + ": " + e.getMessage(), e);
}
@@ -128,26 +128,26 @@ private void initialize() throws IOException {
private void initialize(String subdomain, String token, String roomName, String hudsonUrl, boolean ssl, boolean smartNotify) throws IOException {
campfire = new Campfire(subdomain, token, ssl);
- String exception_message = "Failed to initialize campfire notifier";
+ String exceptionMsg = "Failed to initialize campfire notifier";
try {
this.room = campfire.findRoomByName(roomName);
if ( this.room == null ) {
String roomNotFoundMsg = "Room '" + roomName + "' not found";
- LOGGER.log(Level.WARNING, exception_message, roomNotFoundMsg);
+ LOGGER.log(Level.WARNING, exceptionMsg, roomNotFoundMsg);
throw new IOException(roomNotFoundMsg);
}
} catch (IOException e) {
- LOGGER.log(Level.WARNING, exception_message, e);
- throw new IOException(exception_message, e);
+ LOGGER.log(Level.WARNING, exceptionMsg, e);
+ throw new IOException(exceptionMsg, e);
} catch (ParserConfigurationException e) {
- LOGGER.log(Level.WARNING, exception_message, e);
- throw new IOException(exception_message, e);
+ LOGGER.log(Level.WARNING, exceptionMsg, e);
+ throw new IOException(exceptionMsg, e);
} catch (XPathExpressionException e) {
- LOGGER.log(Level.WARNING, exception_message, e);
- throw new IOException(exception_message, e);
+ LOGGER.log(Level.WARNING, exceptionMsg, e);
+ throw new IOException(exceptionMsg, e);
} catch (SAXException e) {
- LOGGER.log(Level.WARNING, exception_message, e);
- throw new IOException(exception_message, e);
+ LOGGER.log(Level.WARNING, exceptionMsg, e);
+ throw new IOException(exceptionMsg, e);
}
this.hudsonUrl = hudsonUrl;
this.smartNotify = smartNotify;
Please sign in to comment.
Something went wrong with that request. Please try again.