From ec827ab3717bb6634eb4f943af2850adb2f3e953 Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Sat, 31 Aug 2013 02:11:13 +0900
Subject: [PATCH 01/14] Remove unused import statement.

---
 src/main/scala/app/ControllerBase.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/scala/app/ControllerBase.scala b/src/main/scala/app/ControllerBase.scala
index 8d6b21c44..9b6b8dfb2 100644
--- a/src/main/scala/app/ControllerBase.scala
+++ b/src/main/scala/app/ControllerBase.scala
@@ -1,7 +1,7 @@
 package app
 
 import _root_.util.Directory._
-import _root_.util.{StringUtil, FileUtil, Validations}
+import _root_.util.{FileUtil, Validations}
 import org.scalatra._
 import org.scalatra.json._
 import org.json4s._

From 4479ef31e2ce5b33fe858f344558dd444d1e3e5e Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Sat, 31 Aug 2013 02:28:58 +0900
Subject: [PATCH 02/14] (refs #90)Display validation error message for Default
 Branch.

---
 src/main/twirl/settings/options.scala.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/twirl/settings/options.scala.html b/src/main/twirl/settings/options.scala.html
index d386ac372..e41deda34 100644
--- a/src/main/twirl/settings/options.scala.html
+++ b/src/main/twirl/settings/options.scala.html
@@ -20,6 +20,7 @@
                 <option value="@branch"@if(branch==repository.repository.defaultBranch){ selected}>@branch</option>
               }
             </select>
+            <span class="error" id="error-defaultBranch"></span>
           </fieldset>
           <fieldset class="margin">
             <label>

From 897c5ecac72e1d25f3a818b3968a460fc6423ebd Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Sun, 1 Sep 2013 21:15:17 +0900
Subject: [PATCH 03/14] Fix default smtp port in constant.

---
 src/main/scala/service/SystemSettingsService.scala | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/scala/service/SystemSettingsService.scala b/src/main/scala/service/SystemSettingsService.scala
index 40c71ebb9..6d9d2ade6 100644
--- a/src/main/scala/service/SystemSettingsService.scala
+++ b/src/main/scala/service/SystemSettingsService.scala
@@ -47,7 +47,7 @@ trait SystemSettingsService {
       if(getValue(props, Notification, false)){
         Some(Smtp(
           getValue(props, SmtpHost, ""),
-          getOptionValue(props, SmtpPort, Some(25)),
+          getOptionValue(props, SmtpPort, Some(DefaultSmtpPort)),
           getOptionValue(props, SmtpUser, None),
           getOptionValue(props, SmtpPassword, None),
           getOptionValue[Boolean](props, SmtpSsl, None)))
@@ -99,6 +99,7 @@ object SystemSettingsService {
     password: Option[String],
     ssl: Option[Boolean])
 
+  val DefaultSmtpPort = 25
   val DefaultLdapPort = 389
 
   private val AllowAccountRegistration = "allow_account_registration"

From f5520e799120f60fc6e86b2b7a14fcc65b978e24 Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Sun, 1 Sep 2013 21:17:55 +0900
Subject: [PATCH 04/14] (refs #10) Completed notification implementation.

---
 src/main/scala/app/IssuesController.scala     |  11 +-
 .../scala/servlet/TransactionFilter.scala     |  17 +--
 src/main/scala/util/Notifier.scala            | 111 ++++++++++++++----
 3 files changed, 108 insertions(+), 31 deletions(-)

diff --git a/src/main/scala/app/IssuesController.scala b/src/main/scala/app/IssuesController.scala
index fab8b750a..67bbc5b02 100644
--- a/src/main/scala/app/IssuesController.scala
+++ b/src/main/scala/app/IssuesController.scala
@@ -4,7 +4,7 @@ import jp.sf.amateras.scalatra.forms._
 
 import service._
 import IssuesService._
-import util.{CollaboratorsAuthenticator, ReferrerAuthenticator, ReadableUsersAuthenticator}
+import util.{CollaboratorsAuthenticator, ReferrerAuthenticator, ReadableUsersAuthenticator, Notifier}
 import org.scalatra.Ok
 
 class IssuesController extends IssuesControllerBase
@@ -112,7 +112,14 @@ trait IssuesControllerBase extends ControllerBase {
     // record activity
     recordCreateIssueActivity(owner, name, userName, issueId, form.title)
 
-    redirect(s"/${owner}/${name}/issues/${issueId}")
+    val uri = s"/${owner}/${name}/issues/${issueId}"
+
+    // notifications
+    Notifier().toNotify(repository, issueId, form.content.getOrElse("")){
+      Notifier.msgIssue(baseUrl + uri)
+    }
+
+    redirect(uri)
   })
 
   ajaxPost("/:owner/:repository/issues/edit/:id", issueEditForm)(readableUsersOnly { (form, repository) =>
diff --git a/src/main/scala/servlet/TransactionFilter.scala b/src/main/scala/servlet/TransactionFilter.scala
index 5a26734ea..0e9627371 100644
--- a/src/main/scala/servlet/TransactionFilter.scala
+++ b/src/main/scala/servlet/TransactionFilter.scala
@@ -3,7 +3,6 @@ package servlet
 import javax.servlet._
 import org.slf4j.LoggerFactory
 import javax.servlet.http.HttpServletRequest
-import scala.slick.session.Database
 
 /**
  * Controls the transaction with the open session in view pattern.
@@ -21,15 +20,19 @@ class TransactionFilter extends Filter {
       // assets don't need transaction
       chain.doFilter(req, res)
     } else {
-      val context = req.getServletContext
-      Database.forURL(context.getInitParameter("db.url"),
-          context.getInitParameter("db.user"),
-          context.getInitParameter("db.password")) withTransaction {
+      Database(req.getServletContext) withTransaction {
         logger.debug("TODO begin transaction")
         chain.doFilter(req, res)
         logger.debug("TODO end transaction")
       }
     }
   }
-  
-}
\ No newline at end of file
+
+}
+
+object Database {
+  def apply(context: ServletContext): scala.slick.session.Database =
+    scala.slick.session.Database.forURL(context.getInitParameter("db.url"),
+        context.getInitParameter("db.user"),
+        context.getInitParameter("db.password"))
+}
diff --git a/src/main/scala/util/Notifier.scala b/src/main/scala/util/Notifier.scala
index dab8eb320..c8e6821e2 100644
--- a/src/main/scala/util/Notifier.scala
+++ b/src/main/scala/util/Notifier.scala
@@ -1,37 +1,104 @@
 package util
 
-import org.apache.commons.mail.{DefaultAuthenticator, SimpleEmail}
+import scala.concurrent._
+import ExecutionContext.Implicits.global
+import org.apache.commons.mail.{DefaultAuthenticator, HtmlEmail}
+import org.slf4j.LoggerFactory
 
-import service.SystemSettingsService.{SystemSettings, Smtp}
+import app.Context
+import service.{AccountService, RepositoryService, IssuesService, SystemSettingsService}
+import servlet.Database
+import SystemSettingsService.Smtp
 
-trait Notifier {
+trait Notifier extends RepositoryService with AccountService with IssuesService {
+  def toNotify(r: RepositoryService.RepositoryInfo, issueId: Int, content: String)
+      (msg: String => String)(implicit context: Context): Unit
+
+  protected def recipients(issue: model.Issue)(notify: String => Unit)(implicit context: Context) =
+    (
+        // individual repository's owner
+        issue.userName ::
+        // collaborators
+        getCollaborators(issue.userName, issue.repositoryName) :::
+        // participants
+        issue.openedUserName ::
+        getComments(issue.userName, issue.repositoryName, issue.issueId).map(_.commentedUserName)
+    )
+    .distinct
+    .withFilter ( _ != context.loginAccount.get.userName )	// the operation in person is excluded
+    .foreach ( getAccountByUserName(_) foreach (x => notify(x.mailAddress)) )
 
 }
 
 object Notifier {
-  def apply(settings: SystemSettings) = {
-    new Mailer(settings.smtp.get)
+  // TODO We want to be able to switch to mock.
+  def apply(): Notifier = new SystemSettingsService {}.loadSystemSettings match {
+    case settings if settings.notification => new Mailer(settings.smtp.get)
+    case _ => new MockMailer
   }
 
+  def msgIssue(url: String) = (content: String) => s"""
+    |${content}<br/>
+    |--<br/>
+    |<a href="${url}">View it on GitBucket</a>
+    """.stripMargin
+
+  def msgPullRequest(url: String) = (content: String) => s"""
+    |${content}<hr/>
+    |View, comment on, or merge it at:<br/>
+    |<a href="${url}">${url}</a>
+    """.stripMargin
+
+  def msgComment(url: String) = (content: String) => s"""
+    |${content}<br/>
+    |--<br/>
+    |<a href="${url}">View it on GitBucket</a>
+    """.stripMargin
+
+  def msgStatus(id: Int, url: String) = (content: String) => s"""
+    |${content} <a href="${url}">#${id}</a>
+    """.stripMargin
 }
 
-class Mailer(val smtp: Smtp) extends Notifier {
-  def notifyTo(issue: model.Issue) = {
-    val email = new SimpleEmail
-    email.setHostName(smtp.host)
-    email.setSmtpPort(smtp.port.get)
-    smtp.user.foreach { user =>
-      email.setAuthenticator(new DefaultAuthenticator(user, smtp.password.getOrElse("")))
-    }
-    smtp.ssl.foreach { ssl =>
-      email.setSSLOnConnect(ssl)
-    }
-    email.setFrom("TODO address", "TODO　name")
-    email.addTo("TODO")
-    email.setSubject(s"[${issue.repositoryName}] ${issue.title} (#${issue.issueId})")
-    email.setMsg("TODO")
+class Mailer(private val smtp: Smtp) extends Notifier {
+  private val logger = LoggerFactory.getLogger(classOf[Mailer])
 
-    email.send
+  def toNotify(r: RepositoryService.RepositoryInfo, issueId: Int, content: String)
+      (msg: String => String)(implicit context: Context) = {
+    val f = future {
+      val email = new HtmlEmail
+      email.setHostName(smtp.host)
+      email.setSmtpPort(smtp.port.get)
+      smtp.user.foreach { user =>
+        email.setAuthenticator(new DefaultAuthenticator(user, smtp.password.getOrElse("")))
+      }
+      smtp.ssl.foreach { ssl =>
+        email.setSSLOnConnect(ssl)
+      }
+      email.setFrom("notifications@gitbucket.com", context.loginAccount.get.userName)
+      email.setHtmlMsg(msg(view.Markdown.toHtml(content, r, false, true)))
+
+      // TODO Can we use the Database Session in other than Transaction Filter?
+      Database(context.request.getServletContext) withSession {
+        getIssue(r.owner, r.name, issueId.toString) foreach { issue =>
+          email.setSubject(s"[${r.name}] ${issue.title} (#${issueId})")
+          recipients(issue) {
+            email.getToAddresses.clear
+            email.addTo(_).send
+          }
+        }
+      }
+      "Notifications Successful."
+    }
+    f onSuccess {
+      case s => logger.debug(s)
+    }
+    f onFailure {
+      case t => logger.error("Notifications Failed.", t)
+    }
   }
 }
-class MockMailer extends Notifier
\ No newline at end of file
+class MockMailer extends Notifier {
+  def toNotify(r: RepositoryService.RepositoryInfo, issueId: Int, content: String)
+      (msg: String => String)(implicit context: Context): Unit = {}
+}
\ No newline at end of file

From 3fc792fcf87c790b35941630911587973284b2e8 Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Mon, 2 Sep 2013 07:40:41 +0900
Subject: [PATCH 05/14] (refs #10) Add notification. Notification of when
 merging is not implemented yet.

---
 src/main/scala/app/IssuesController.scala     | 53 +++++++++++--------
 .../scala/app/PullRequestsController.scala    |  8 ++-
 src/main/scala/util/Notifier.scala            |  4 +-
 3 files changed, 39 insertions(+), 26 deletions(-)

diff --git a/src/main/scala/app/IssuesController.scala b/src/main/scala/app/IssuesController.scala
index 67bbc5b02..adee1e867 100644
--- a/src/main/scala/app/IssuesController.scala
+++ b/src/main/scala/app/IssuesController.scala
@@ -112,14 +112,12 @@ trait IssuesControllerBase extends ControllerBase {
     // record activity
     recordCreateIssueActivity(owner, name, userName, issueId, form.title)
 
-    val uri = s"/${owner}/${name}/issues/${issueId}"
-
     // notifications
     Notifier().toNotify(repository, issueId, form.content.getOrElse("")){
-      Notifier.msgIssue(baseUrl + uri)
+      Notifier.msgIssue(s"${baseUrl}/${owner}/${name}/issues/${issueId}")
     }
 
-    redirect(uri)
+    redirect(s"/${owner}/${name}/issues/${issueId}")
   })
 
   ajaxPost("/:owner/:repository/issues/edit/:id", issueEditForm)(readableUsersOnly { (form, repository) =>
@@ -136,21 +134,15 @@ trait IssuesControllerBase extends ControllerBase {
 
   post("/:owner/:repository/issue_comments/new", commentForm)(readableUsersOnly { (form, repository) =>
     handleComment(form.issueId, Some(form.content), repository)() map { case (issue, id) =>
-      if(issue.isPullRequest){
-        redirect(s"/${repository.owner}/${repository.name}/pull/${form.issueId}#comment-${id}")
-      } else {
-        redirect(s"/${repository.owner}/${repository.name}/issues/${form.issueId}#comment-${id}")
-      }
+      redirect(s"/${repository.owner}/${repository.name}/${
+        if(issue.isPullRequest) "pull" else "issues"}/${form.issueId}#comment-${id}")
     } getOrElse NotFound
   })
 
   post("/:owner/:repository/issue_comments/state", issueStateForm)(readableUsersOnly { (form, repository) =>
     handleComment(form.issueId, form.content, repository)() map { case (issue, id) =>
-      if(issue.isPullRequest){
-        redirect(s"/${repository.owner}/${repository.name}/pull/${form.issueId}#comment-${id}")
-      } else {
-        redirect(s"/${repository.owner}/${repository.name}/issues/${form.issueId}#comment-${id}")
-      }
+      redirect(s"/${repository.owner}/${repository.name}/${
+        if(issue.isPullRequest) "pull" else "issues"}/${form.issueId}#comment-${id}")
     } getOrElse NotFound
   })
 
@@ -289,9 +281,10 @@ trait IssuesControllerBase extends ControllerBase {
       val (action, recordActivity) =
         getAction(issue)
           .collect {
-            case "close" if(issue.isPullRequest)  => true  -> (Some("close")  -> Some(recordClosePullRequestActivity _))
-            case "close" if(!issue.isPullRequest) => true  -> (Some("close")  -> Some(recordCloseIssueActivity _))
-            case "reopen"                         => false -> (Some("reopen") -> Some(recordReopenIssueActivity _))
+            case "close"  => true  -> (Some("close")  ->
+                Some(if(issue.isPullRequest) recordClosePullRequestActivity _ else recordCloseIssueActivity _))
+            case "reopen" => false -> (Some("reopen") ->
+                Some(recordReopenIssueActivity _))
           }
           .map { case (closed, t) =>
             updateClosed(owner, name, issueId, closed)
@@ -307,15 +300,29 @@ trait IssuesControllerBase extends ControllerBase {
           }
 
       // record activity
-      content foreach { content =>
-        if(issue.isPullRequest)
-          recordCommentPullRequestActivity(owner, name, userName, issueId, content)
-        else
-          recordCommentIssueActivity(owner, name, userName, issueId, content)
+      content foreach {
+        (if(issue.isPullRequest) recordCommentPullRequestActivity _ else recordCommentIssueActivity _)
+        (owner, name, userName, issueId, _)
       }
       recordActivity foreach ( _ (owner, name, userName, issueId, issue.title) )
 
-      (issue, commentId)
+      // notifications
+      Notifier() match {
+        case f =>
+          content foreach {
+            f.toNotify(repository, issueId, _){
+              Notifier.msgComment(s"${baseUrl}/${owner}/${name}/${
+                if(issue.isPullRequest) "pull" else "issues"}/${issueId}#comment-${commentId}")
+            }
+          }
+          action foreach {
+            f.toNotify(repository, issueId, _){
+              Notifier.msgStatus(s"${baseUrl}/${owner}/${name}/issues/${issueId}")
+            }
+          }
+      }
+
+      issue -> commentId
     }
   }
 
diff --git a/src/main/scala/app/PullRequestsController.scala b/src/main/scala/app/PullRequestsController.scala
index 0cd143fd6..add3cdad8 100644
--- a/src/main/scala/app/PullRequestsController.scala
+++ b/src/main/scala/app/PullRequestsController.scala
@@ -1,6 +1,6 @@
 package app
 
-import util.{LockUtil, CollaboratorsAuthenticator, JGitUtil, ReferrerAuthenticator}
+import util.{LockUtil, CollaboratorsAuthenticator, JGitUtil, ReferrerAuthenticator, Notifier}
 import util.Directory._
 import util.Implicits._
 import service._
@@ -303,8 +303,14 @@ trait PullRequestsControllerBase extends ControllerBase {
         .call
     }
 
+    // record activity
     recordPullRequestActivity(repository.owner, repository.name, loginUserName, issueId, form.title)
 
+    // notifications
+    Notifier().toNotify(repository, issueId, form.content.getOrElse("")){
+      Notifier.msgPullRequest(s"${baseUrl}/${repository.owner}/${repository.name}/pull/${issueId}")
+    }
+
     redirect(s"/${repository.owner}/${repository.name}/pull/${issueId}")
   })
 
diff --git a/src/main/scala/util/Notifier.scala b/src/main/scala/util/Notifier.scala
index c8e6821e2..b9ed57995 100644
--- a/src/main/scala/util/Notifier.scala
+++ b/src/main/scala/util/Notifier.scala
@@ -55,8 +55,8 @@ object Notifier {
     |<a href="${url}">View it on GitBucket</a>
     """.stripMargin
 
-  def msgStatus(id: Int, url: String) = (content: String) => s"""
-    |${content} <a href="${url}">#${id}</a>
+  def msgStatus(url: String) = (content: String) => s"""
+    |${content} <a href="${url}">#${url split('/') last}</a>
     """.stripMargin
 }
 

From bdc06feb88cce201ca3272bbd6b988f0334b07dd Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Tue, 3 Sep 2013 20:58:38 +0900
Subject: [PATCH 06/14] Fix a problem in pull request to branches other than
 the master branch.

---
 src/main/scala/app/PullRequestsController.scala | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/scala/app/PullRequestsController.scala b/src/main/scala/app/PullRequestsController.scala
index add3cdad8..bad9b7ecd 100644
--- a/src/main/scala/app/PullRequestsController.scala
+++ b/src/main/scala/app/PullRequestsController.scala
@@ -100,7 +100,7 @@ trait PullRequestsControllerBase extends ControllerBase {
       getPullRequest(repository.owner, repository.name, issueId).map { case (issue, pullreq) =>
         val remote = getRepositoryDir(repository.owner, repository.name)
         val tmpdir = new java.io.File(getTemporaryDir(repository.owner, repository.name), s"merge-${issueId}")
-        val git    = Git.cloneRepository.setDirectory(tmpdir).setURI(remote.toURI.toString).call
+        val git = Git.cloneRepository.setDirectory(tmpdir).setURI(remote.toURI.toString).setBranch(pullreq.branch).call
 
         try {
           // mark issue as merged and close.
@@ -179,7 +179,7 @@ trait PullRequestsControllerBase extends ControllerBase {
         FileUtils.deleteDirectory(tmpdir)
       }
 
-      val git = Git.cloneRepository.setDirectory(tmpdir).setURI(remote.toURI.toString).call
+      val git = Git.cloneRepository.setDirectory(tmpdir).setURI(remote.toURI.toString).setBranch(branch).call
       try {
         git.checkout.setName(branch).call
 

From d588531ab8e7551484b959690533c4900f552f63 Mon Sep 17 00:00:00 2001
From: Tomofumi Tanaka <tanacasino@gmail.com>
Date: Wed, 4 Sep 2013 00:11:13 +0900
Subject: [PATCH 07/14] Make configurable data(git,db) dir using env vars

---
 src/main/scala/servlet/AutoUpdateListener.scala | 1 +
 src/main/scala/util/Directory.scala             | 7 ++++++-
 src/main/webapp/WEB-INF/web.xml                 | 5 -----
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/main/scala/servlet/AutoUpdateListener.scala b/src/main/scala/servlet/AutoUpdateListener.scala
index 51f8c2c5d..308dfcaac 100644
--- a/src/main/scala/servlet/AutoUpdateListener.scala
+++ b/src/main/scala/servlet/AutoUpdateListener.scala
@@ -112,6 +112,7 @@ class AutoUpdateListener extends org.h2.server.web.DbStarter {
   private val logger = LoggerFactory.getLogger(classOf[AutoUpdateListener])
   
   override def contextInitialized(event: ServletContextEvent): Unit = {
+    event.getServletContext.setInitParameter("db.url", s"jdbc:h2:${Directory.DatabaseHome}")
     super.contextInitialized(event)
     logger.debug("H2 started")
     
diff --git a/src/main/scala/util/Directory.scala b/src/main/scala/util/Directory.scala
index 754d73728..73cee55aa 100644
--- a/src/main/scala/util/Directory.scala
+++ b/src/main/scala/util/Directory.scala
@@ -7,11 +7,16 @@ import java.io.File
  */
 object Directory {
 
-  val GitBucketHome = new File(System.getProperty("user.home"), "gitbucket").getAbsolutePath
+  val GitBucketHome = (scala.util.Properties.envOrNone("GITBUCKET_HOME") match {
+    case Some(env) => new File(env)
+    case None => new File(System.getProperty("user.home"), "gitbucket")
+  }).getAbsolutePath
 
   val GitBucketConf = new File(GitBucketHome, "gitbucket.conf")
   
   val RepositoryHome = s"${GitBucketHome}/repositories"
+
+  val DatabaseHome = s"${GitBucketHome}/data"
   
   /**
    * Repository names of the specified user.
diff --git a/src/main/webapp/WEB-INF/web.xml b/src/main/webapp/WEB-INF/web.xml
index 901034601..77d4c18f2 100644
--- a/src/main/webapp/WEB-INF/web.xml
+++ b/src/main/webapp/WEB-INF/web.xml
@@ -55,11 +55,6 @@
     <listener-class>servlet.AutoUpdateListener</listener-class>
   </listener>
   
-  <context-param>
-    <param-name>db.url</param-name>
-    <param-value>jdbc:h2:~/gitbucket/data</param-value>
-  </context-param>
-  
   <context-param>
     <param-name>db.user</param-name>
     <param-value>sa</param-value>

From d84d40afea6d7cb647796e72c0ce1cf9102fac2a Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Wed, 4 Sep 2013 02:19:04 +0900
Subject: [PATCH 08/14] Set init parameters using
 ServletContext#setInitParameter().

---
 .../scala/servlet/GitRepositoryServlet.scala   | 18 +++---------------
 1 file changed, 3 insertions(+), 15 deletions(-)

diff --git a/src/main/scala/servlet/GitRepositoryServlet.scala b/src/main/scala/servlet/GitRepositoryServlet.scala
index 94c432be5..eac94c4a7 100644
--- a/src/main/scala/servlet/GitRepositoryServlet.scala
+++ b/src/main/scala/servlet/GitRepositoryServlet.scala
@@ -24,21 +24,9 @@ class GitRepositoryServlet extends GitServlet {
   
   override def init(config: ServletConfig): Unit = {
     setReceivePackFactory(new GitBucketReceivePackFactory())
-    
-    // TODO are there any other ways...?
-    super.init(new ServletConfig(){
-      def getInitParameter(name: String): String = name match {
-        case "base-path"  => Directory.RepositoryHome
-        case "export-all" => "true"
-        case name => config.getInitParameter(name)
-      }
-      def getInitParameterNames(): java.util.Enumeration[String] = {
-        config.getInitParameterNames
-      }
-      
-      def getServletContext(): ServletContext = config.getServletContext
-      def getServletName(): String = config.getServletName
-    });
+    config.getServletContext.setInitParameter("base-path", Directory.RepositoryHome)
+    config.getServletContext.setInitParameter("export-all", "true")
+    super.init(config)
   }
   
 }

From 3fc0fa5a02307bedb1e5aad469b2cdaaed8721db Mon Sep 17 00:00:00 2001
From: takezoe <takezoe@gmail.com>
Date: Wed, 4 Sep 2013 03:39:28 +0900
Subject: [PATCH 09/14] Don't set response content type via Accept header.

---
 src/main/scala/app/ControllerBase.scala | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/scala/app/ControllerBase.scala b/src/main/scala/app/ControllerBase.scala
index 9b6b8dfb2..11c0d8b8a 100644
--- a/src/main/scala/app/ControllerBase.scala
+++ b/src/main/scala/app/ControllerBase.scala
@@ -22,9 +22,8 @@ abstract class ControllerBase extends ScalatraFilter
 
   implicit val jsonFormats = DefaultFormats
 
-//  before() {
-//    contentType = "text/html"
-//  }
+  // Don't set content type via Accept header.
+  override def format(implicit request: HttpServletRequest) = ""
 
   override def doFilter(request: ServletRequest, response: ServletResponse, chain: FilterChain) {
     val httpRequest  = request.asInstanceOf[HttpServletRequest]

From 3c0681d55de4f7df1ae7a9070df599751bb534ad Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Wed, 4 Sep 2013 10:40:44 +0900
Subject: [PATCH 10/14] (refs #10) Add notification of when merged.

---
 src/main/scala/app/PullRequestsController.scala | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/main/scala/app/PullRequestsController.scala b/src/main/scala/app/PullRequestsController.scala
index bad9b7ecd..3a287f395 100644
--- a/src/main/scala/app/PullRequestsController.scala
+++ b/src/main/scala/app/PullRequestsController.scala
@@ -155,6 +155,11 @@ trait PullRequestsControllerBase extends ControllerBase {
             }
           }
 
+          // notifications
+          Notifier().toNotify(repository, issueId, "merge"){
+            Notifier.msgStatus(s"${baseUrl}/${repository.owner}/${repository.name}/pull/${issueId}")
+          }
+
           redirect(s"/${repository.owner}/${repository.name}/pull/${issueId}")
 
         } finally {

From d60695992b729e0bad0ff83166cda7c0ec9a3003 Mon Sep 17 00:00:00 2001
From: Naoki Takezoe <takezoe@gmail.com>
Date: Wed, 4 Sep 2013 11:27:14 +0900
Subject: [PATCH 11/14] Update README.md

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index 8b93b51f1..b5bb03308 100644
--- a/README.md
+++ b/README.md
@@ -45,6 +45,7 @@ Release Notes
 - Add the branch tab in the repository viewer.
 - Encoding auto detection for the file content in the repository viewer.
 - Add favicon, header logo and icons for the timeline.
+- Specify data directory via environment variable GITBUCKET_HOME.
 - Fixed some bugs.
 
 ### 1.4 - 31 Jul 2013

From 420ca85393b4204b2e43f8e7e61f840fab099bfd Mon Sep 17 00:00:00 2001
From: shimamoto <chibochibo@gmail.com>
Date: Wed, 4 Sep 2013 18:45:37 +0900
Subject: [PATCH 12/14] (refs #10) Except group account in notification.

---
 src/main/scala/util/Notifier.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/scala/util/Notifier.scala b/src/main/scala/util/Notifier.scala
index b9ed57995..4fc522f92 100644
--- a/src/main/scala/util/Notifier.scala
+++ b/src/main/scala/util/Notifier.scala
@@ -26,7 +26,7 @@ trait Notifier extends RepositoryService with AccountService with IssuesService
     )
     .distinct
     .withFilter ( _ != context.loginAccount.get.userName )	// the operation in person is excluded
-    .foreach ( getAccountByUserName(_) foreach (x => notify(x.mailAddress)) )
+    .foreach ( getAccountByUserName(_) filterNot (_.isGroupAccount) foreach (x => notify(x.mailAddress)) )
 
 }
 

From f89afc175f07bdf1091dbee31bc794a260b3eb08 Mon Sep 17 00:00:00 2001
From: Naoki Takezoe <takezoe@gmail.com>
Date: Wed, 4 Sep 2013 18:45:11 +0900
Subject: [PATCH 13/14] Update README.md

---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index b5bb03308..5ebb9721a 100644
--- a/README.md
+++ b/README.md
@@ -37,7 +37,7 @@ To upgrade GitBucket, only replace gitbucket.war.
 
 Release Notes
 --------
-### 1.5 - COMMING SOON!
+### 1.5 - 4 Sep 2013
 - Fork and pull request.
 - LDAP authentication.
 - Mail notification.

From c7776b5b3714baf2e86ed1f5e21e55276d17b9ff Mon Sep 17 00:00:00 2001
From: Naoki Takezoe <takezoe@gmail.com>
Date: Wed, 4 Sep 2013 18:46:09 +0900
Subject: [PATCH 14/14] Update README.md

---
 README.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 5ebb9721a..1c211e1cb 100644
--- a/README.md
+++ b/README.md
@@ -10,17 +10,17 @@ The current version of GitBucket provides a basic features below:
 - Repository search (Code and Issues)
 - Wiki
 - Issues
+- Fork / Pull request
+- Mail notification
 - Activity timeline
 - User management (for Administrators)
 - Group (like Organization in Github)
 
 Following features are not implemented, but we will make them in the future release!
 
-- Fork and pull request
 - Network graph
 - Statics
 - Watch / Star
-- Notification
 
 If you want to try the development version of GitBucket, see the documentation for developers at [Wiki](https://github.com/takezoe/gitbucket/wiki).