You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@esme.apache.org by es...@apache.org on 2012/04/06 14:50:19 UTC
svn commit: r1310306 - in /esme/trunk/server: ./ src/main/scala/
src/main/scala/org/apache/esme/api/ src/main/scala/org/apache/esme/external/
src/main/scala/org/apache/esme/model/
Author: esjewett
Date: Fri Apr 6 12:50:18 2012
New Revision: 1310306
URL: http://svn.apache.org/viewvc?rev=1310306&view=rev
Log:
Turn on deprecation warnings and fix all warnings on the 'sbt compile' step. Warnings still exist for test classes.
Modified:
esme/trunk/server/build.sbt
esme/trunk/server/src/main/scala/JettyLauncher.scala
esme/trunk/server/src/main/scala/org/apache/esme/api/API2.scala
esme/trunk/server/src/main/scala/org/apache/esme/external/RssFeed.scala
esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala
esme/trunk/server/src/main/scala/org/apache/esme/model/UserConvFollow.scala
esme/trunk/server/src/main/scala/org/apache/esme/model/UserTagFollow.scala
Modified: esme/trunk/server/build.sbt
URL: http://svn.apache.org/viewvc/esme/trunk/server/build.sbt?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/build.sbt (original)
+++ esme/trunk/server/build.sbt Fri Apr 6 12:50:18 2012
@@ -8,7 +8,10 @@ version := "1.4"
organization := "Apache Software Foundation"
-scalaVersion := "2.9.1"
+scalaVersion := "2.9.1"
+
+//scalacOptions ++= Seq("-unchecked", "-deprecation")
+scalacOptions ++= Seq("-deprecation")
seq(webSettings :_*)
@@ -76,6 +79,4 @@ resolvers += "Maven central" at "http://
// Execute tests in the current project serially.
// Tests from other projects may still run concurrently.
-//parallelExecution in Test := false
-
-//scalacOptions += "-Xprint:typer"
+parallelExecution in Test := false
\ No newline at end of file
Modified: esme/trunk/server/src/main/scala/JettyLauncher.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/JettyLauncher.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/JettyLauncher.scala (original)
+++ esme/trunk/server/src/main/scala/JettyLauncher.scala Fri Apr 6 12:50:18 2012
@@ -22,7 +22,7 @@
import org.eclipse.jetty.server.nio.SelectChannelConnector
import net.liftweb.http.LiftFilter
- object JettyLauncher extends Application {
+ object JettyLauncher extends App {
val port = if(System.getenv("PORT") != null) System.getenv("PORT").toInt else 8080
val server = new Server
val scc = new SelectChannelConnector
Modified: esme/trunk/server/src/main/scala/org/apache/esme/api/API2.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/api/API2.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/api/API2.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/api/API2.scala Fri Apr 6 12:50:18 2012
@@ -250,7 +250,7 @@ object API2 extends ApiHelper with XmlHe
} yield {
val token: AuthToken = AuthToken.create
.user(user)
- .description(S.param("description"))
+ .description(S.param("description").openOr(""))
.saveMe
(200,Map(),Full(tokenToXml(token)))
}
@@ -351,7 +351,7 @@ object API2 extends ApiHelper with XmlHe
millis,
xml match {
case Full(x) => xml
- case _ => Box({new Atom(S.param("metadata") openOr "")})
+ case _ => Full(new Atom(S.param("metadata").openOr("")))
},
from,
S.param("replyto").map(toLong),
Modified: esme/trunk/server/src/main/scala/org/apache/esme/external/RssFeed.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/external/RssFeed.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/external/RssFeed.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/external/RssFeed.scala Fri Apr 6 12:50:18 2012
@@ -26,7 +26,7 @@ import org.apache.esme.model.User
import org.apache.esme.model.Message
import org.apache.esme.actor.Distributor.{UserCreatedMessage=>Msg}
-import net.liftweb.common.Empty
+import net.liftweb.common.{Empty,Full,Box}
object RssFeed {
val dateFormats = List(new SimpleDateFormat("EEE, dd MMM yyyy HH:mm:ss Z", US))
@@ -86,17 +86,16 @@ class RssFeed(user: User, rssURL: String
*/
override def getLastSortedMessages(msgs: List[Msg], lastMessage: Option[Msg]): List[Msg] = {
- import net.liftweb.common.Box
lastMessage match {
case Some(message: Msg) =>
// a hack to format text identically- difference in urls & trailing whitespace
val lastMessageText =
Message.create.setTextAndTags(message.text, Nil, Empty).
- choice((m: Message) => Box(m.body.trim))(Box("")).get
+ choice((m: Message) => Full(m.body.trim))(Full("")).get
//get.body.trim
msgs.takeWhile{ msg =>
Message.create.setTextAndTags(msg.text, Nil, Empty).
- choice((m: Message) => Box(m.body.trim))(Box("")).get != lastMessageText
+ choice((m: Message) => Full(m.body.trim))(Full("")).get != lastMessageText
//get.body.trim != lastMessageText
}
case None => msgs
Modified: esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/model/UserAuth.scala Fri Apr 6 12:50:18 2012
@@ -128,7 +128,7 @@ trait FieldSet {
object UserPwdAuthModule extends AuthModule {
def loginPresentation: Box[NodeSeq] = {
val ldapBind : CssBindFunc = "#ldapEnabled [value]" #> (Props.getBool("ldap.enabled") openOr false)
- TemplateFinder.findAnyTemplate("templates-hidden" :: "upw_login_form" :: Nil) match {
+ Templates("templates-hidden" :: "upw_login_form" :: Nil) match {
case Full(tpl) => Full(ldapBind(tpl))
case _ => Empty
}
@@ -177,7 +177,7 @@ object UserPwdAuthModule extends AuthMod
private var email = ""
def toForm: NodeSeq =
- TemplateFinder.findAnyTemplate("templates-hidden" :: "upw_signup_form" :: Nil).map(
+ Templates("templates-hidden" :: "upw_signup_form" :: Nil).map(
xhtml =>
bind("signup", xhtml,
"email" -%> SHtml.text(email, s => email = s.trim.toLowerCase),
@@ -218,8 +218,7 @@ object UserPwdAuthModule extends AuthMod
}
object OpenIDAuthModule extends AuthModule {
- def loginPresentation: Box[NodeSeq] =
- TemplateFinder.findAnyTemplate("templates-hidden" :: "openid_login_form" :: Nil)
+ def loginPresentation: Box[NodeSeq] = Templates("templates-hidden" :: "openid_login_form" :: Nil)
def moduleName: String = "openid"
@@ -328,7 +327,7 @@ trait LDAPBase {
myLdapVendor
}
- @deprecated
+ @deprecated("Use getAttributesMap.","01-01-2012")
def getAttrs(dn : String) : Map[String, List[String]] = {
var attrsMap = Map.empty[String, List[String]]
val attributes = getLDAPAttributes(dn)
@@ -501,7 +500,8 @@ object ContainerManagedAuthModule extend
}
case Empty => {
S.error(S.?("base_user_err_unknown_creds"))
- }
+ }
+ case _ => {}
}
S.redirectTo(from)
@@ -520,7 +520,7 @@ object LDAPAuthModule extends AuthModule
override def isDefault = false
- def loginPresentation: Box[NodeSeq] = TemplateFinder.findAnyTemplate("templates-hidden" :: "ldap_login_form" :: Nil)
+ def loginPresentation: Box[NodeSeq] = Templates("templates-hidden" :: "ldap_login_form" :: Nil)
def moduleName: String = "ldap"
@@ -551,10 +551,11 @@ object LDAPAuthModule extends AuthModule
//TODO: There's no corresponding property in User's Mapper
//ldapAttrs.get("mail").flatMap(_.headOption.map(usr))
debug("Attributes from LDAP for user '%s'. Firstname: '%s', lastname: '%s'".format(name, usr.firstName, usr.lastName))
- usr.save
+ usr.save
})
UserAuth.create.authType(moduleName).user(usr).authKey(name).save
logInUser(usr)
+ case _ => {}
}
} else {
S.error(S.?("base_user_err_unknown_creds"))
Modified: esme/trunk/server/src/main/scala/org/apache/esme/model/UserConvFollow.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/model/UserConvFollow.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/model/UserConvFollow.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/model/UserConvFollow.scala Fri Apr 6 12:50:18 2012
@@ -27,6 +27,6 @@ object UserConvFollow extends UserConvFo
class UserConvFollow extends LongKeyedMapper[UserConvFollow] with IdPK {
def getSingleton = UserConvFollow
- object user extends LongMappedMapper(this, User)
- object conversation extends LongMappedMapper(this, Message)
+ object user extends MappedLongForeignKey(this, User)
+ object conversation extends MappedLongForeignKey(this, Message)
}
\ No newline at end of file
Modified: esme/trunk/server/src/main/scala/org/apache/esme/model/UserTagFollow.scala
URL: http://svn.apache.org/viewvc/esme/trunk/server/src/main/scala/org/apache/esme/model/UserTagFollow.scala?rev=1310306&r1=1310305&r2=1310306&view=diff
==============================================================================
--- esme/trunk/server/src/main/scala/org/apache/esme/model/UserTagFollow.scala (original)
+++ esme/trunk/server/src/main/scala/org/apache/esme/model/UserTagFollow.scala Fri Apr 6 12:50:18 2012
@@ -27,6 +27,6 @@ object UserTagFollow extends UserTagFoll
class UserTagFollow extends LongKeyedMapper[UserTagFollow] with IdPK {
def getSingleton = UserTagFollow
- object user extends LongMappedMapper(this, User)
- object tag extends LongMappedMapper(this, Tag)
+ object user extends MappedLongForeignKey(this, User)
+ object tag extends MappedLongForeignKey(this, Tag)
}
\ No newline at end of file