Errata

entityManager.create(cust)
should be
entityManager.persist(cust)

Variables cruisePk and cabinPk are referenced as cruisePK and cabinPK, respectively, on the following page (28).

entityManager.create(reservation);
should be
entityManager.persist(reservation);

In the "Message-Driven Beans" section that starts on page 27, the example code, which continues through the top of page 28, contains an error.

On page 28, it states:

Reservation reservation = new Reservation(customer, cruise, cabin, price, new Date());

entityManager.create(reservation);

However, in the Java EE 5 SDK, the interface 'EntityManager' does not contain a method "create(Object)".

"The <name> element represents the set of classes" ...
should describe the name attribute, not element; <name> is not an element in the XML
appearing above the text (on the previous page), it's an attribute on the
persistence-unit element.

There is no "dev" directory of titan.jar, so the first box should be removed or
renamed to "/" (the root of the file). Also, a box with the name "META-INF" should be
added to the root folder and list the file "persistence.xml" as contents.

p. 58:
Object ref = jndiContext.lookup("TravelAgentBean/remote");
TravelAgentRemote dao = (TravelAgentRemote)
PortableRemoteObject.narrow(ref,TravelAgentRemote.class);

p. 60:
Object ref = jndiContext.lookup("TravelAgentRemote");
CabinHomeRemote home = (TravelAgentRemote)
PortableRemoteObject.narrow(ref,TravelAgentRemote.class);

Not sure about this, but I think the "CabinHomeRemote" interface on page 60 is left
over from an earlier edition of the book (and EJB spec). I think it should be
"TravelAgentRemote", as it is on page 58.

Object ref = jndiContext.lookup("TravelAgentRemote");

should be:

Object ref = jndiContext.lookup("TravelAgentBean/remote");

There is no "dev" directory of titan-persistence.jar, so the first box should be
removed or renamed to "/" (the root of the file).

the second sentence, "TransactionRequiredException is thrown if this method
is invoked on a transaction-scoped persistence context." should be
"TransactionRequiredException is thrown if this method is NOT invoked on a
transaction-scoped persistence context."

According to the API doc, the paragraph should read like:
getReference() differs from find() in that if the entity is not found in the database, the EntityNotFoundException is thrown when the entity's state is first accessed. The persistence provider runtime may also throw the EntityNotFoundException when getReference is called.

There hasn't been introduced any methods named "getResource()", so what the author probably
means is the "getReference()" method of the entity manager.

Should be "The TransactionRequiredException is thrown if this method is NOT invoked
on a transaction-scoped persistence context."

Should be "The TransactionRequiredException is thrown if this method is NOT invoked on a transaction-scoped persistence context."

Should be "The TransactionRequiredException is thrown if this method is NOT invoked
on a transaction-scoped persistence context."

Change:
"We also want firstName to have a not-null contraint"
To:
"We also want firstName and lastName to have a not-null contraint".

create table CUSTOMER_TABLE
{

}

The code example must read:

create table CUSTOMER_TABLE
{
CUST_ID integer primary key not null,
FIRST_NAME varchar(20) not null,
lastName varchar(255)
}

to match the Customer entity class below. Then the text
"We also want firstName to have a not-null constraint and want ..."
does not have to be changed.

@Id annotation has no generator attribute, @GeneratedValue has a generator attribute to specify the primary key generator

So the text must be: The name() attribute defines the the name of the @TableGenerator and is the name referenced in the @GeneratedValue.generator() attribute.

The name of the sequence generator has to be specified in the @GeneratedValue annotation, not in the @Id.

So the text must be: The name() attribute specifies how this @SequenceGenerator is referenced in the @GeneratedValue annotation.

<id-class> doesn't use embedded text to set the class name. It uses the class attribute
instead. So instead of

<id-class>com.titan.domain.CustomerPK</id-class>

it should read

<id-class class="com.titan.domain.CustomerPK"/>

public PK getPk()...
should be
public CustomerPK getPk()...

There is probably a ")" missing in
@AttributeOverride(name="lastName"....LAST_NAME"),
before the comma

public PK getPk()...
should probably read
public CustomerPK getPk()...

The <transient> element is used within an <attributes> element, not within a <attribute> element.

private Address address;
should be
private Address homeAddress;
for the get/set functions to work.

<cascade>ALL</cascade>
should be
<cascade-all/>

In the method getCreditCard(), "return homeAddress;" should be "return creditCard;".

PHONE table foreign key should be CUSTOMER_ID rather than CREDIT_CARD_ID

attribute targetEntity should be target-entity.

attribute targetEntity should be target-entity.

@JoinTable(name="CUSTOMER_PHONE"),
should be
@JoinTable(name="CUSTOMER_PHONE",

The code snippets and text on this and other pages throughout the book refer to a
user-provided no-arg constructor as the "default constructor". This is wrong as the
Java Language Specification describes the default constructor as the (no argument)
constructor automatically provided if the class does not contain any (user-provided)
constructors.

I suggest reviewing the text (and code samples) and replacing "default constructor"
and "default no-arg constructor" with "no-arg constructor" as appropriate.

The "date" property should be annotated with the
@Temporal(TemporalType.DATE) annotation.

<one-to-many name="ship"
should be
<one-to-many name="reservations"

The method "public void setCustomers(Set customers);" should be
changed to "public void setCustomers(Set<Customer> customers)"
and have a method body like "{ this.customers = customers; }".

<many-to-many name="cruise"
should be
<many-to-many name="reservations"

The method
"public void setCustomers(Set customers);"
should be changed to
"public void setCustomers(Set<Customer> customers)".

A current correction has already stated that the method should have a
method body like on p144.

should setCustomers(Set customers)
be
setCustomers(List<Customer> customers)
or
should the text explain why set it being used

This is the first example of a List-based collection.

Customer cust is defined but Customer customer is used instead.

"customer.setAddress(new Address());"
should be
"cust.setAddress(new Address());"

"customer.setPhoneNumbers().add(new Phone());"
should be
"cust.setPhoneNumbers().add(new Phone());"

declaration needs opening brace

public @interface DiscriminatorColumn
public @interface DiscriminatorColumn {

@AttributeOverride(name="lastname",...
should be
@AttributeOverride(name="lastName",...

</entity> tag after <mapped-superclass> is superfluous

entityManager.creatQuery( should be entityManager.createQuery(

In the first line of the first code example on page 172 it should read

"Query query = entityManager.createQuery("

instead of

"Query query = entityManager.creatQuery(" .

SELECT res FROM ...
should be
SELECT cust FROM ...

Change "Reservation EJBs" to "Reservation entities".

The paragraph mentions CMR fields and CMP fields which don't exist in EJB3.

In the code example, replace "SELECT DISTINCT COUNT(c.address.zip)" with
"SELECT COUNT(DISTINCT c.address.zip)".

...(SELECT res FROM cr.reservations WHERE ...
should be
...(SELECT res FROM cr.reservations res WHERE ...

<entity-result entity-class="com.titan.domain.CreditCard"/>
should be
<entity-result entity-class="com.titan.domain.CreditCard">

@SqlResultSetMapping(name="reservationCount",
entities=@EntityResult(name="com.titan.domain.Cruise",
fields=@FieldResult(name="id",column="id")),
columns=@ColumnResult(name="resCount"))

should be

@SqlResultSetMapping(name="reservationCount",
entities={@EntityResult(entityClass=Cruise.class,
fields={@FieldResult(name="id",column="id")})},
columns={@ColumnResult(name="resCount")})

I disagree with the unconfirmed errata:

@SqlResultSetMapping(name="reservationCount",
entities=@EntityResult(name="com.titan.domain.Cruise",
fields=@FieldResult(name="id",column="id")),
columns=@ColumnResult(name="resCount"))

is correct, because the {} can be omitted if only one element of an array is specified.

Change
Query.setParameter("cruise", cruise);
to
query.setParameter("cruise", cruise);

result-set-mapping="customerAndCCNumMapping"/>

should be

result-set-mapping="customerAndCCNumMapping">

afterInsert() should read as beforeInsert()

beforeInsert() should read as afterInsert()

(218pp) Headline starting "Domain objects";
The book uses the term "domain object" to denote types of objects that
typically are "transfer objects" (or in some cases possibly "value objects")
in the sense of Patterns of Enterprise Applications (Fowler).

While I understand the desire to avoid terms that are too close to
"data transfer object" as used in EJB 2.x contexts, I believe that the choice
of "domain object" is unfortunate in this case.

The EJB 3.0 spec states: "An entity is a lightweight persistent domain object."
However, the author uses the term to refer to a non-persistent object. This
might be confusing to the non-expert reader. Therefore, I suggest changing
the terminology in future versions of the book.

Change:
The EJB name defaults to the unqualified name of the bean class if you
initialize this attribute.

To:
The EJB name defaults to the unqualified name of the bean class if you
don't specify this attribute.

In the xml listing on pages 224 and 225,
the "env-entry" should be specified before "resource-ref"
(jboss rejects it as listed).

In the code sample, the last two lines should be changed from:

public EJBHome getEJBHome()
java.lang.IllegalStateException;
public EJBLocalHome getEJBLocalHome()
java.lang.IllegalStateException;

to

public EJBHome getEJBHome()
throws java.lang.IllegalStateException;
public EJBLocalHome getEJBLocalHome()
throws java.lang.IllegalStateException;

In the withdraw method is the principal not initialized. The following line should be added

java.security.Principal pricipal = context.getCallerPrincipal();

The updateAddress() method should check to see if (customer == null) and
throw an IncompleteConversationalState() exception if it is.

In the first code sample, change:

mapMsg.setInt("CustomerID", ticket.customerID.intValue());
mapMsg.setInt("CruiseID", ticket.cruiseID.intValue());
mapMsg.setInt("CabinID", ticket.cabinID.intValue());

to

mapMsg.setInt("CustomerID", ticket.customerID);
mapMsg.setInt("CruiseID", ticket.cruiseID);
mapMsg.setInt("CabinID", ticket.cabinID);

Change "javax.jta.UserTransaction" to "javax.transaction.UserTransaction".

There seem to be a lot of mistakes with the xml listing:

1. line 7: </resource-ref> should by </ejb-local-ref>

2. line 18: </env-entry> should be </persistence-context-ref>

2. lines 20,21:

<resource-ref-name> should be <res-ref-name>

<resource-type> should be <res-type>

<injection-target-name> should be connectionFactory instead of datasource

<injection-target-class> should be com.titan.travelagent.TravelAgentBean and not
javax.jms.Topic

In the second code sample, replace the second occurrence of "implements" with a comma.

Replace "implement" with "implements"

Change

ShipMaintenanceRemote maintenance =
(ShipMaintenanceRemote) jndiCntxt.lookup("ShipMaintenanceRemote ");
ShipMaintenanceRemote Calendar april2nd = Calendar.getInstance();

to

ShipMaintenanceRemote maintenance =
(ShipMaintenanceRemote) jndiCntxt.lookup("ShipMaintenanceRemote");
Calendar april2nd = Calendar.getInstance();

3rd parameter of scheduleMaintenance method,
Date dateOfTest
should be
Date dateOf

Remove non-relevant "@PersistenceContext(unitName="titanDB") entityManager;"

Change:
String item = ship + " is scheduling maintenance of " + description;
To:
String msg = ship + " is scheduling maintenance of " + description;

wrong parameter names.
correct
"timerService.createTimer(dateOf, msg);"
to
"timerService.createTimer(dateOfTest, item);"

line:
@Resource(mappedName="MaintenanceTopic") topic;
should be
@Resource(mappedName="MaintenanceTopic") Topic topic;

Date dateOfTest
should be
Date dateOf

Body of the "scheduleMaintenance(String ship, String description, Date dateOfTest)" method:
Correct
"timerService.createTimer(dateOf, item);"
to
"timerService.createTimer(dateOfTest, item);"

2nd "@Resource" annotation without type specified.
Correct
"@Resource(mappedName="MaintenanceTopic") topic"
to
"@Resource(mappedName="MaintenanceTopic") Topic topic"

The line:
long ninetyDays = 1000 * 60 * 60 * 24 * 90; // 90 days
Is incorrect. The int values are multiplied using integer math, which overflows before it is assigned to a long.
long ninetyDays = 1000L * 60 * 60 * 24 * 90; // 90 days
behaves properly.

Other long calculations on the next page happen to fit in an int.

Replace the paragraph

"The scheduleMaintenance() method first obtains a Collection of all timers
defined for the ship."

to
"The scheduleMaintenance() method first obtains a Collection of all timers
defined for the ShipMaintenanceBean."

replace:
Timer.cancel();
with:
timer.cancel();

Contrary to the contents of the section, the Timer.getNextTimeout() method
always returns a java.util.Date representing the time at which the timer
expires next, and Timer.getTimeRemaining() returns the interval until expiration.
In other words, the text "If, however, the timer is an interval timer, the Date
returned is the time remaining until the next expiration." should be removed.

Replace "javax.ejb.NoSuchObjectException" with "javax.ejb.NoSuchLocalObjectException".

getTimer() also throws java.lang.IllegalStateException

@PostCreate
should be
@PostConstruct

The implements is dropped in the class definition of StatelessTimerBean.

It should be

StatelessTimerBean implements javax.ejb.TimedObject

@PostCreate
should be
@PostConstruct

"enterprise naming context" should be "environment naming context"
(as per Java EE 5 and EJB 3 specs).

This mistake is found throughout the book. However, the correct name is
given in a few instances.

@javax.annotation.EJB must be @javax.ejb.EJB

The given xml code for the ejb-jar.xml is not valid. The <session> tag is not closed.

The xml tag <ejb-local-ref> is opened but the xml tag </ejb-ref> is closed. This is
not valid xml.

The <ejb-local-ref> element defines a reference to remote EJBs.
should be
The <ejb-local-ref> element defines a reference to local EJBs.

<ejb-ref-name>ejb/ProcessPaymentRemote</ejb-ref-name>

should be

<ejb-ref-name>ejb/ProcessPaymentLocal</ejb-ref-name>

Change

ProcessPaymentLocal local = (ProcessPaymentRemote)
jndiContext.lookup("java:comp/env/ejb/ProcessPaymentLocal");

to

ProcessPaymentLocal local = (ProcessPaymentLocal)
jndiContext.lookup("java:comp/env/ejb/ProcessPaymentLocal");

Replace "inventory.jar" with "inventory-ejb.jar".

In the code sample, replace

EntityManager titan = (EntityManagerFactory)
jndiContext.lookup("java:comp/env/ejb/persistence/TitanDB");

with

EntityManager titan = (EntityManager)
jndiContext.lookup("java:comp/env/ejb/persistence/TitanDB");

EntityManager titan = (EntityManagerFactoy) jndiContext.lookup("java:comp/env/persistence/TitanDB")

==>

EntityManager titan = (EntityManager) jndiContext.lookup("java:comp/env/persistence/TitanDB")

Add missing "</persistence-context-ref>" closing element to XML.

Change:
type=javax.sql.DataSource,
To:
type=javax.sql.DataSource.class,

Change:
type=javax.sql.DataSource,
To:
type=javax.sql.DataSource.class,

Change:
type=javax.jms.Topic,
To:
type=javax.jms.Topic.class,

the method should return a value,
likely, it should end with:

return returnValue;

the custom annotation (from the previous code block, and used in the following page) is named
'JndiIndected'.

the annotation used here is '@JndiInject',

text should read:
@JndiInjected("java:/TransactionManager")

utx.begin();
...
utx.commit();

should be

tran.begin();
...
tran.commit();

Change:
"javax.ejb.TransactionManagementType of Bean using the @javax.ejb.TransactionManager
annotation"

To:

"javax.ejb.TransactionManagementType of BEAN using the @javax.ejb.TransactionManagement
annotation".

Change:
@TransactionManagement(TransactionManagerType.BEAN)
To:
@TransactionManagement(TransactionManagementType.BEAN)

Hello,

Please find chapter 16, table 16-1, page 401, CMT with transaction type attributes: Reqired, Mandatory, Supports, System exception.
At Container's action there is only info that container rethrows javax.ejb.EJBTransactionRolledbackExeption.
According to Sun Micosystems JSR 220: Enterprise JaaBeans, ver 3.0, EJB Core Contracts and Requiremnts, table 14, page 360 and 361 the javax.transaction.TransactionRollbackException may be thrown to the client. Please find note D, pag 361.

Client may receives javax.ejb.EJBTransactionRolledbackException and
javax.transaction.TransactionRollbackException.

Kind Regards
Witold Koszulinski

Hello,

Accordng to Sun Mcrosystems spec. .., chapter 14 Excepion handling, 14.3.4 Exceptions from Message-Driven Bean Message Listener Methods,table 19, Bean method runs in the context of the tansaction that..., for System Exceptions row, Container's action column contains "Throw EJBException that wraps the original exception to resource adapter". This is omited in table 16-2 Exception summary formessage-deriven beans, page 403, first row, transaction attrbute = Required, System Exception, Container's action.

Please consider this in your errata.

Kind Regards
Witold Koszulinski

In the code sample, replace:
@RolesAllowed("AUTOHORIZED_TRAVEL_AGENT")
with:
@RolesAllowed("AUTOHORIZED_MERCHANT")

The AUTHORIZED_MERCHANT role seems to be called AUTHORIZED_TRAVEL_AGENT in the
preceding text and (annotation-based) example. This causes annotation-related text
and example to be inconsistent with the XML-related text and example.

Change:
<method-name>byCheck</method-name>
To:
<method-name>byCash</method-name>

In the XML fragment, change

<method-params>double[]</method-params>

to

<method-params>
<method-param>double[]</method-param>
</method-params>

Change:
<user-caller-identity/>
To:
<use-caller-identity/>

Page 422:
javax.security.Principal getCallerPrincipal();
should be
java.security.Principal getCallerPrincipal();

import javax.security.Principal;
should be
import java.security.Principal;

[exercise19_1] build.xml target name="run.client";
the system property "java.endorsed.dirs" has to be set into the run.client target of build.xml
for ex19_1 and other related examples like below, otherwise the client will run in a
unmarshalling error:

<target name="run.client" depends="clientjar">
<java classname="com.titan.clients.Client" fork="yes" dir=".">
<classpath refid="classpath"/>
<sysproperty key="java.endorsed.dirs" value="<jboss-home>libendorsed"/>
</java>
</target>

In the second column, change

Byte
Boolean
Short
Int
Long
float
Double

to

byte
boolean
short
int
long
float
double

It's explained in the following paragraph when it is the object wrapper classes that are used.

Change:
The address attribute
To:
The location attribute

(or change to "The <soap:address> element").

Seems that there is something wrong with the listing. Two EntityManagers with the
same variable name "em" are declared. One @PersistenceContext(unitname="titanDB")
private EntityManager em; and one @PersistenceContext EntityManager em;

The constructor for the javax.xml.ws.Service does not take a String, but a URL
object.

The TravelAgent and Customer diagrams should have "<<Entity>>" added to them.
Remove the arrow between the TravelAgent and the Customer entities.

Same corrections as in figure 21-4.

Same arrow correction as in figure 21-4.

"LEFT FETCH JOIN" should be "LEFT JOIN FETCH"

It's probably a recent change, but www.jboss.org has the downloads page that used to be available at www.jboss.com.

http://jakarta.apache.org/ant must be http://ant.apache.org

<persistence-unit>
<name>titan</name>

should be:

<persistence-unit name="titan">

In the JBoss example available for download, the PortableRemoteObject.narrow() has been
removed. This also applies to many other examples that are inconsistent.

Change:
<persistence-unit name="titan">
To:
<persistence-unit name="titan" transaction-type="RESOURCE_LOCAL">

Add the following method:

public void setTimeCreated(Date date) { timeCreated = date; }

public PK getPk()...
should probably read
public CustomerPK getPk()...

The consumer application is very similar to the client application in Exercise 12.1.

should read

The consumer application is very similar to the client application class JmsClient_1 in Chapter 12.

Start Up JBoss must refer to Workbook 1 instead of Chapter 15

"$JBOSS_HOME/service/default/deploy" should be "$JBOSS_HOME/server/default/deploy"