Jean-Francois Leveque

Comments in English

...@@ -5,51 +5,50 @@ import org.legrog.entities.*; ...@@ -5,51 +5,50 @@ import org.legrog.entities.*;
5 import java.util.List; 5 import java.util.List;
6 6
7 /** 7 /**
8 - * Service partagé pour les données non associées à un package spécifique. 8 + * Shared service for data not part of a specific package
9 */ 9 */
10 public interface SharedService { 10 public interface SharedService {
11 11
12 /** 12 /**
13 - * Ajoute un pays. 13 + * Adds a Country
14 * 14 *
15 - * @param country 15 + * @param country Country to add
16 - * @return Country ajouté 16 + * @return Country
17 */ 17 */
18 Country addCountry(Country country); 18 Country addCountry(Country country);
19 19
20 /** 20 /**
21 - * Fournit la liste des pays. 21 + * Provides the List of Country
22 * 22 *
23 * @return List<Country> 23 * @return List<Country>
24 */ 24 */
25 List<Country> getAllCountries(); 25 List<Country> getAllCountries();
26 26
27 /** 27 /**
28 - * Fournit la liste des DisplayNameMask 28 + * Provides the List of DisplayNameMask
29 * 29 *
30 * @return List<DisplayNameMask> 30 * @return List<DisplayNameMask>
31 */ 31 */
32 List<DisplayNameMask> getAllDisplayNameMasks(); 32 List<DisplayNameMask> getAllDisplayNameMasks();
33 33
34 /** 34 /**
35 - * Fournit la liste des AccountRole 35 + * Provides the List of AccountRole
36 * 36 *
37 * @return List<AccountRole> 37 * @return List<AccountRole>
38 */ 38 */
39 List<AccountRole> getAvailableUserRoles(); 39 List<AccountRole> getAvailableUserRoles();
40 40
41 /** 41 /**
42 - * Fournit la liste des AccountProperty 42 + * Provides the List of AccountProperty
43 * 43 *
44 * @return List<AccountProperty> 44 * @return List<AccountProperty>
45 */ 45 */
46 List<AccountProperty> getAvailableUserProperties(); 46 List<AccountProperty> getAvailableUserProperties();
47 47
48 /** 48 /**
49 - * Fournit le compte correspondant à l'utilisateur courant. 49 + * Provides the Account of the current user
50 * @return Account 50 * @return Account
51 * 51 *
52 - * TODO : implémenter une méthode qui le fait vraiment
53 */ 52 */
54 Account getCurrentUser(); 53 Account getCurrentUser();
55 } 54 }
...\ No newline at end of file ...\ No newline at end of file
......
...@@ -25,10 +25,10 @@ public class SharedServiceDefault implements SharedService { ...@@ -25,10 +25,10 @@ public class SharedServiceDefault implements SharedService {
25 /** 25 /**
26 * Uses shared repositories for country and related to account and also AccountService 26 * Uses shared repositories for country and related to account and also AccountService
27 * 27 *
28 - * @param countryRepository 28 + * @param countryRepository injected CountryRepository
29 - * @param accountRoleRepository 29 + * @param accountRoleRepository injected accountRoleRepository
30 - * @param accountPropertyRepository 30 + * @param accountPropertyRepository injected AccountPropertyRepository
31 - * @param accountService 31 + * @param accountService injected AccountService
32 */ 32 */
33 @Inject 33 @Inject
34 public SharedServiceDefault(CountryRepository countryRepository, AccountRoleRepository accountRoleRepository, 34 public SharedServiceDefault(CountryRepository countryRepository, AccountRoleRepository accountRoleRepository,
...@@ -74,7 +74,7 @@ public class SharedServiceDefault implements SharedService { ...@@ -74,7 +74,7 @@ public class SharedServiceDefault implements SharedService {
74 74
75 @Override 75 @Override
76 public Account getCurrentUser() { 76 public Account getCurrentUser() {
77 - // TODO Remplacer l'astuce par une vraie récupération de l'utilisateur 77 + // TODO Implement instead of randomly choosing one
78 List<Account> accounts = accountService.getAllUsers(); 78 List<Account> accounts = accountService.getAllUsers();
79 Random random = new Random(); 79 Random random = new Random();
80 80
......