Annotate classes with @FilterWith annotation... currently it's only possible on method level

• messageformat documentation
• not important for example:
  public Result getCount(Context ctx) {
  return Results.json(count.get());
  }
• "annotation" is wrong must be validation
  annotation.hasViolations()
• Write documentation for @required and other stuff...
  @required dokumentieren...

add to documentation that this works nicely and parses json easily: ublic Result postNatureUserIdsAndGetUrls(List postRequestNatureUserIds) {

• variable parts
• injecting stuff

I guess the streaming and especially the concatenation can be made faster.
Well. it will be cached by Freemarker. Therefore the impact might be minmal

in TemplateEngineFreemarkerEscapedLoader

=> Gwt resty interface
=> ... more stuff...

is missing => write one...

• get url from controller method.

Like in play1 or in django:
csrf https://docs.djangoproject.com/en/dev/ref/contrib/csrf/#how-csrf-works

=> a form must have a token that can be validated

Like exemplified in Play 1 and other frameworks.

Something like
Lang.setLang(en) or so...

(proposed by James)

Problem: making url parsing better:
which brings us to the topic of ninja... it uses getServletPath() and getPathInfo(), which both return decoded strings, which at some point can/will cause problems

And a close improvement: Use a tree structure
what it should probably do is parse the URI itself, and parse the routes itself, and do segment by segment matching, on decoded segments. this actually would allow a significant performance optimisation too, since, with 60 routes, it won't have to run 60 regular expressions, it can store the routes in a tree structure based on common path segments, and just do equals on those

If the result does not have a content type the type is inferred via:
if (result.getContentType() == null) {
if (context.getRequestContentType() != null) {
result.contentType(context.getRequestContentType());
in ResultHandler.java.

That is wrong as it calls:
return httpServletRequest.getContentType(); in ContextImpl.

this returns the content type.. and that might be text/html; encoding=utf-8 ...
wrong...

better would be to have
context.getAcceptContentType()
context.getAcceptEncoding()

context.getRequestContentType() should be marked as deprecated... or (may be better) documented in a better way... it simply returns the request content type including encoding...

Solution: We have to parse the "Accept" header (case insensitive).

One idea. Play 1 uses the following code that only allows html json xml as content types. Opinionated but makes parsing of the Accept header simple. Source code below.

But there might be better ways to solve this as Ninja potentially allows to support custom content types as well.

    /**
     * Automatically resolve request format from the Accept header
     * (in this order : html > xml > json > text)
     */
    public void resolveFormat() {

        if (format != null) {
            return;
        }

        if (headers.get("accept") == null) {
            format = "html".intern();
            return;
        }

        String accept = headers.get("accept").value();

        if (accept.indexOf("application/xhtml") != -1 || accept.indexOf("text/html") != -1 || accept.startsWith("*/*")) {
            format = "html".intern();
            return;
        }

        if (accept.indexOf("application/xml") != -1 || accept.indexOf("text/xml") != -1) {
            format = "xml".intern();
            return;
        }

        if (accept.indexOf("text/plain") != -1) {
            format = "txt".intern();
            return;
        }

        if (accept.indexOf("application/json") != -1 || accept.indexOf("text/javascript") != -1) {
            format = "json".intern();
            return;
        }

        if (accept.endsWith("*/*")) {
            format = "html".intern();
            return;
        }
    }

Solution 1)

• Better documentation
  Solution 2)
• add it to ninjaproperties to make the usage similar everywhere...

One for jcache and one for memchached. Should be straight forward. Good examples are Play 1's classes.

This code in LangImpl will be not fast:

@OverRide
public Map<Object, Object> getAll(String language) {

    Configuration configuration = getLanguageConfigurationForLocale(language);

    return ConfigurationConverter.getMap(configuration);

}

mvn ninja:run

• based on jetty 8+
• minimal version of jetty (as fast as the 6 series)
• does reloads automatically

module java missing

Otherwise it is not nice to test...

eg makes a "vanityUrl" filter quite simple to implement

for instance

@OverRide
public Result filter(FilterChain filterChain, Context context) {
return filterChain.next(new WrappedContext(context) {
@OverRide
public String getPathParameter(String key) {
if (USER_ID_SESSION_KEY.equals(key)) {
return username;
} else {
return super.getPathParameter(key);
}

To do something like mvn ninja:new

• mybatis migratoins
• http://code.google.com/p/flyway/wiki/HowDoesFlywayWork
• carbon migrations

crypto toolkit to sign client side cookie - should allow to automatically invalidate session client side... => save timestamp into cookie...

=> make templating engine exchangable

• maybe as generic as replacing json with html renderer...

• nginx scaling
• apache reverse scaling

let ninja generate the secret automatically into application.conf

• assets
• module
• for instance ebeans

Add object to map conversion for freemarker...

freemarker itself is a bit picky...

simply use jackson (or beanutils) to convert any object into a map....

=> write an example for content negotiation

Using the MessageFormat of ICU for J it should be really
simple to get a perfect i18n plural support...

http://icu-project.org/apiref/icu4j/com/ibm/icu/text/MessageFormat.html

Demo the writing and reading of databases in project ninja-demo-application-app-engine.

• possibly with objectify (preferred)
• possibly with siena
• possibly with possibly

• Use cache by default
  https://devcenter.heroku.com/articles/increasing-application-performance-with-http-cache-headers
• Use hash to invalidate assets by default
  https://devcenter.heroku.com/articles/rack-cache-memcached-static-assets-rails31
• Make a TagHelper that generates link to asset from FreemarkerTemplate

Something like Logger().log...

=> must be there and tested