Commit graph

494 commits

Author SHA1 Message Date
michaelisvy
77bd960b69 Merge pull request #98 from freemansoft/master
Add comments to describe when AOP invoked
2015-10-14 05:17:54 +08:00
Antoine Rey
4bb829cbd1 Fix #101 display the pet type when using the JDBC profile 2015-10-13 18:41:18 +02:00
Antoine Rey
cb0504ee97 #92 add some comments to switch from HSQLDB to MySQL 2015-10-13 08:32:15 +02:00
Tomas Repel
f7498c79df Removed HTTP PUT method - it is not supported in JSP 2.3 2015-10-02 11:33:57 +02:00
Joe Freeman
5a6c108efb Added comments when CallMonitoringAspect called
CallMonitoringAspect is not invoked when spring-data-jpa is used.  Spent
4 hours trying to understand why jmx bean wasn't showing message counts.
Use JPA or JDBC if you want call counts exposed via JMX / AOP
2015-09-30 21:20:34 -04:00
Antoine Rey
06be7eb5be #92 Use column alias 2015-08-20 08:19:57 +02:00
Antoine Rey
91d19d17b8 #92 Fix column 'visits.id' not found on MySql 2015-08-19 18:55:52 +02:00
Antoine Rey
0fdd1504b4 Removing deprecated javadoc 2015-06-29 09:03:58 +02:00
Antoine Rey
7fe5184bc8 Servlet 3.0, JSP 2.2 and EL 2.2 are the minimum required to deploy Petclinic 2015-06-29 09:01:08 +02:00
Antoine Rey
8b625617cb #87 Petclinic should be compatible with Java 7 for the time being 2015-06-29 08:50:04 +02:00
Antoine Rey
80ff54ac03 Fix #89 Web layer: use @Valid whenever possible 2015-06-29 08:37:29 +02:00
Antoine Rey
98d9bbb629 Removing reference to the unknown UserResource class 2015-06-24 18:43:45 +02:00
Antoine Rey
71f2424542 Remove explicit unboxing 2015-06-24 08:12:43 +02:00
Antoine Rey
6f6fa64dd7 Remove unused VisitRepository from constructor 2015-06-24 08:11:09 +02:00
Antoine Rey
ca3bb07fa4 Remove unused WebApplicationContext property 2015-06-18 18:29:49 +02:00
Antoine Rey
1b4d4256c4 Fix typo into javadoc 2015-06-18 18:28:31 +02:00
Antoine Rey
735fb1149b Remove explicit unboxing 2015-06-17 21:07:10 +02:00
michaelisvy
dc0fb9abd8 removing unused method #85 2015-06-17 13:10:00 -05:00
Antoine Rey
5570366cfd Use a simple RowMapper instead of a BeanPropertyRowMapper 2015-06-17 08:30:26 +02:00
michaelisvy
8d20340518 removed unused attribute #64 2015-06-09 17:14:53 +08:00
Antoine Rey
fb64465802 Add some javadoc 2015-06-09 08:40:49 +02:00
Antoine Rey
818529b5b1 #64 Remove N+1 select by using the OneToManyResultSetExtractor of Spring Data Core JDBC Extensions 2015-06-09 08:37:20 +02:00
michaelisvy
cb6bd875de Putting encoding filter first per #80 2015-05-29 15:20:10 +08:00
michaelisvy
72dc0c882f Merge pull request #48 from 7footmoustache/patch-1
Fixed some typos in comments
2015-05-20 23:28:00 +08:00
michaelisvy
83301cb602 Centering content 2015-05-20 23:21:37 +08:00
michaelisvy
54b7d87ff2 Fixing log back warning per #59 2015-05-20 23:18:19 +08:00
michaelisvy
c00d1abf23 Merge pull request #72 from ka2m/master
Update JDBC file naming in MySQL setup instruction
2015-05-20 23:14:45 +08:00
michaelisvy
c553d39f8d Merge pull request #67 from mklose/patch-1
fix typo
2015-05-20 23:13:04 +08:00
michaelisvy
c8e360298b fixed bug: vets.html did not display properly 2015-05-13 01:07:46 +09:00
michaelisvy
91ed548481 removing xdd version number
so we always use the latest in the classpath
2015-05-12 19:55:15 +08:00
michaelisvy
4c859ed512 minor update on comment 2015-05-12 19:52:00 +08:00
michaelisvy
80c1d243a1 Removed RSS/rome and added JSon
RSS is becoming outdated, JSon is a better example
2015-05-12 19:07:35 +08:00
michaelisvy
9914056a81 removing test that causes lots if issues
Happens since introduction of new mvc namespace tag for
content-negotiation
2015-05-10 06:45:39 +08:00
michaelisvy
694390d0c2 migrated assertion to assertJ 2015-05-10 06:33:10 +08:00
Vlad Selpukhin
099b848b62 Update JDBC file naming in MySQL steup instruction 2015-04-23 15:43:48 +03:00
michaelisvy
38a5b28e29 Improving/fixing comments 2015-04-14 09:27:41 +08:00
mklose
3f2d3ba72a fix typo 2015-04-01 16:39:51 +02:00
michaelisvy
cc4ae966c6 removed deprecated Mapper in Jdbc 2015-01-21 18:22:46 +08:00
michaelisvy
d8a2b5c737 cleanup on tests 2015-01-20 22:25:24 +08:00
michaelisvy
1dfc3b7a3a Migrated test assertions to AssertJ 2015-01-20 22:02:34 +08:00
michaelisvy
35a179f88b latest versions and imports cleanup 2015-01-20 21:21:27 +08:00
michaelisvy
3e4512781e Sonar review: made code more readable 2015-01-16 09:25:36 +08:00
michaelisvy
1c9b401248 cleaned up if statement in controller 2015-01-16 09:22:40 +08:00
michaelisvy
5c9ab6bd06 test methods:used should/shouldNot 2015-01-16 09:08:22 +08:00
michaelisvy
7080682d3e workaround because there seems to be a conflict
There seems to be a conflict between the following:

1) New MVC namespace syntax:
<mvc:content-negotiation use-not-acceptable="true">
	    	<mvc:default-views>
	      		<bean class="org.springframework.web.servlet.view.JstlView">
	      			<property name="url" value="" />
	      		</bean>
	    	</mvc:default-views>
	  	</mvc:content-negotiation>

2) Inside my Unit test:
ResultActions actions =
this.mockMvc.perform(get("/vets.xml").accept(MediaType.TEXT_XML));
        actions.andDo(print()); // action is logged into the console

        actions.andExpect(status().isOk());
2014-12-05 13:41:55 +08:00
michaelisvy
d7b100f90d Removed 'Help' from menu
We were planning to create a ‘help’ but it never happened
2014-11-28 13:54:31 +09:00
michaelisvy
49c39b65dc renamed headTag.jsp -> staticFiles.jsp 2014-11-28 13:52:09 +09:00
michaelisvy
85c8237fb5 added comment on <mvc:default-servlet-handler /> 2014-11-28 13:40:46 +09:00
michaelisvy
6e079b23bb Error control on description was missing
Took that opportunity to use custom tag (14 lines versus 2 lines)
2014-11-28 13:25:17 +09:00
michaelisvy
f0bf692767 Moving "visit" object to request scope
Previously was in session scope (doesn’t scale as well)
2014-11-28 13:19:25 +09:00