Skip to content
This repository was archived by the owner on Aug 10, 2023. It is now read-only.

Commit 9624ccc

Browse files
committed
Incorporated feedback received on the expert's alias.
1 parent c4d8069 commit 9624ccc

File tree

6 files changed

+12
-10
lines changed

6 files changed

+12
-10
lines changed

spec/chapters/applications.tex

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ \section{\mvc\ Applications}
2727
The path in the application's URL space in which \mvc\ controllers live must
2828
be specified either using the \ApplicationPath\ annotation on the application
2929
subclass or in the web.xml as part of the \code{url-pattern} element. \mvc\
30-
applications MUST use a non-empty path or pattern: i.e., {\em "/"} or {\em "/*"} are
31-
invalid in \mvc\ applications \assertref{application-path}.
30+
applications SHOULD use a non-empty path or pattern: i.e., {\em "/"} or {\em "/*"}
31+
should be avoided whenever possible.
3232

3333
The reason for this is that \mvc\ implementations
3434
often forward requests to the Servlet container, and the use of these

spec/chapters/assertions.tex

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ \chapter{Summary of Assertions}
1414
\code{ValidationResult} field or property is defined.
1515
\assert{event-firing} All events in \code{javax.mvc.event} must be fired. See Javadoc for more
1616
information on each event in that package.
17-
\assert{application-path} The application's path must be non-empty.
1817
\assert{builtin-engines} Implementations must provide support for JSPs and Facelets.
1918
\assert{extension-engines} CDI beans that implement \code{javax.mvc.engine.ViewEngine} provide
2019
an extension mechanism for view engines.

spec/chapters/controllers.tex

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ \section{Controllers}
99

1010
An {\em MVC controller} is a \jaxrs\ \cite{jaxrs} resource method decorated by
1111
an \Controller\ annotation \assertref{controller}.
12-
If this annotation is applied to a class, then all methods in it are regarded as controllers
12+
If this annotation is applied to a class, then all resource methods in it are regarded as controllers
1313
\assertref{all-controllers}.
1414
Using the \Controller\ annotation on a subset of methods defines a hybrid class in which
1515
certain methods are controllers and others are traditional \jaxrs\ resource methods.
@@ -75,6 +75,9 @@ \section{Controllers}
7575
}
7676
\end{listing}
7777

78+
The \View\ annotation only applies to a method that returns void and
79+
it is ignored in all the other cases.
80+
7881
Note that, even though controller methods return types are restricted as
7982
explained above, \mvc\ does not impose any restrictions on parameter
8083
types available to controller methods: i.e., all parameter types injectable

spec/chapters/intro.tex

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ \chapter{Introduction}
1313
the majority of the controller logic is provided by the framework instead of the application.
1414

1515
The API defined by this specification falls into the action-based
16-
category and is, therefore, not intended to be a replacement for component-based framworks such as
16+
category and is, therefore, not intended to be a replacement for component-based frameworks such as
1717
JavaServer Faces (JSF) \cite{jsf}, but simply a different approach to building Web applications on the
1818
Java EE platform.
1919

@@ -46,7 +46,7 @@ \section{Non-Goals}
4646

4747
It is worth noting that, even though a standalone implementation of \mvc\ that runs outside
4848
of Java EE is a non-goal, this specification shall not intentionally prevent implementations
49-
to run in other environments, provided that those environments include support all the
49+
to run in other environments, provided that those environments include support for all the
5050
EE technologies required by \mvc.
5151

5252
\section{Additional Information}
@@ -81,8 +81,8 @@ \section{Terminology}
8181
\label{terminology}
8282

8383
Most of the terminology used in this specification is borrowed from other specifications
84-
such as \jaxrs\ and CDI. We use the terms {\em per-request} and {\em request scoped} as
85-
well as {\em per-application} and {\em application scoped} interchangeably.
84+
such as \jaxrs\ and CDI. We use the terms {\em per-request} and {\em request-scoped} as
85+
well as {\em per-application} and {\em application-scoped} interchangeably.
8686

8787
\section{Conventions}
8888

spec/chapters/validation.tex

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ \section{Exception Mappers}
4141
An application can handle the exception by including an exception mapper as follows:
4242

4343
\begin{listing}{1}
44-
class FormViolationMapper implements
45-
ExceptionMapper<ConstraintViolationException> {
44+
public class FormViolationMapper
45+
implements ExceptionMapper<ConstraintViolationException> {
4646

4747
@Inject
4848
private ErrorDataBean error;

spec/spec.pdf

-222 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)