Commit graph

221 commits

Author SHA1 Message Date
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
michaelisvy
92e7ab43be Simplified ContentNegoViewResolver config
- Now using namespace, much simpler (11 lines instead of 27)
- Just a minor issue with:
<bean class="org.springframework.web.servlet.view.JstlView">
	<property name="url" value="" />
</bean>
(should investigate later if we can remove “url” attribute
2014-11-28 12:45:57 +09:00
michaelisvy
475f5f5349 migrated to Spring 4.1.1.RELEASE
Also had to make changes because Spring now uses com.rometools for RSS
feeds
See here for more details: https://jira.spring.io/browse/SPR-11893
2014-10-18 23:36:15 +08:00
michaelisvy
74f683aaa9 Fixed typo 2014-10-18 22:13:01 +08:00
Craig Dennis
de73bdce6b Fixed some typos in comments 2014-07-07 21:19:40 -07:00
michaelisvy
ce50e80736 Merge pull request #42 from bram-atmire/documentation-fix
update db_readme.txt to reflect properties rename
2014-06-02 08:41:51 +08:00
Thibault Duchateau
e99c67a4bb Upgraded Dandelion-Datatables to the latest release (v0.10.0)
* The old datatables-servlet2 dependency has been removed from the pom.xml
 * Some attributes have been renamed (e.g. paginate => pageable)
 * Starting from the v0.10.0, all web components (servlet, filter) must be explicitely declared in the web.xml file. That's why I added the following declaration: DandelionFilter, DandelionServlet and DatatablesFilter
 * Added a new datatables.properties file in order to disable the asset management brought by Dandelion-Core. This is equivalent to the old cdn=true table attribute
2014-05-31 11:02:32 +02:00
Bram Luyten
1341c12d1f update db_readme.txt to reflect properties rename
jdbc.properties was renamed to data-access.properties, but this file still referred to the old name.
2014-05-11 18:37:42 +02:00
Mic
3f0bfbbdbe updates logo 2014-04-12 23:28:53 +08:00
Mic
622291aba7 Merge branch 'master' of https://github.com/SpringSource/spring-petclinic 2014-04-12 11:27:53 +08:00
michaelisvy
7dc6da25d8 Merge pull request #39 from arey/patch-1
Remove public to methods from the ClinicService
2014-04-12 11:27:32 +08:00
Mic
5e329e1766 Merge branch 'master' of https://github.com/SpringSource/spring-petclinic 2014-04-11 21:28:09 +08:00
Antoine Rey
67ae72f324 Remove public to methods from the ClinicService
Any method you define in an interface is by definition public
2014-03-23 17:43:58 +01:00
Mic
35a94eaa5d migrating to Spring 4.0.2 2014-03-03 21:24:29 +08:00
Antoine Rey
076a124e0a Fixes #37 with other default locale than english
Test in error with french default Locale:
org.junit.ComparisonFailure: expected:<[ne peut pas être vide]> but was:<[may not be empty]>
	at org.junit.Assert.assertEquals(Assert.java:115)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at org.springframework.samples.petclinic.model.ValidatorTests.emptyFirstName(ValidatorTests.java:39)
2014-02-24 14:05:02 +01:00
Mic
08d84ed2b3 added comment for welcome file 2013-12-16 21:00:51 +09:00
Mic
5cfd482edc removed default profile and replaced with "jpa" 2013-12-16 20:58:15 +09:00
Mic
3e8829f372 adding explicit reference to default profile because it doesn't seem to work on on some environments 2013-12-13 08:15:10 +08:00
Mic
34d8ca46ac chaining validation so we can see multiple error messages when there are multiple validation errors 2013-11-06 09:18:33 +08:00
Mic
c4b9b0b119 migrated to the latest version of jquery and jquery-ui 2013-11-06 08:58:15 +08:00
Mic
b7764e39b2 Added comments to explain how database dialect is configured #33 2013-09-27 09:33:09 +08:00
Cyrille Le Clerc
e15e45b5e2 Spring MVC should server static resources (*.html, ...) located in src/main/webapp instead of returning "404 Not Found" 2013-08-10 19:49:01 +02:00
Mic
4c01d60b36 removing Serializable because it creates a lot of warnings
App should be as simple as possible for beginners
2013-07-25 16:35:37 +02:00
michaelisvy
6412e4805b Merge pull request #25 from boly38/patch-1
Update BaseEntity.java
2013-07-20 14:04:13 -07:00