refactor applied to ignore case in search by name

AdrianoSaadeh · AdrianoSaadeh · commit 00aab0622958 · 2020-10-08T19:33:32.000-03:00
diff --git a/src/main/java/com/example/apiRest/dto/FilmsDTO.java b/src/main/java/com/example/apiRest/dto/FilmsDTO.java
@@ -23,7 +23,7 @@ public class FilmsDTO implements Serializable {
 
     @ApiModelProperty(value = "Field is string")
     @NotBlank(message = "Field name can not be blank")
-    @Length(min = 3,max = 50,message = "The field name must have between 3 and 50 character ")
+    @Length(min = 3, max = 50, message = "The field name must have between 3 and 50 character ")
     private String name;
 
     @ApiModelProperty(value = "Field is string")
@@ -51,7 +51,7 @@ public String getName() {
     }
 
     public void setName(String name) {
-        this.name = name;
+        this.name = name.toLowerCase();
     }
 
     public String getGenre() {
@@ -69,10 +69,11 @@ public int getYear() {
     public void setYear(int year) {
         this.year = year;
     }
+
     @Override
     public String toString() {
         return "FilmsDTO [id=" + ", nome=" + name + ", genre=" + genre
-                + ", year=" + year +  "]";
+                + ", year=" + year + "]";
     }
 
 }
diff --git a/src/main/java/com/example/apiRest/service/FilmsService.java b/src/main/java/com/example/apiRest/service/FilmsService.java
@@ -32,7 +32,8 @@ public List<DetailsFilmsDTOResponse> getAllFilms() {
     }
 
     public List<DetailsFilmsDTOResponse> getFilmByName(String nameFilm) {
-        films = filmsRepository.findByName(nameFilm);
+        films = filmsRepository.findByName(nameFilm.toLowerCase());
+        System.out.println(films);
         if (films.isEmpty()) {
             throw new FilmsDoesNotExistException("There are no films that meet this search");
         }
diff --git a/src/main/resources/data.sql b/src/main/resources/data.sql
@@ -1,5 +1,5 @@
-INSERT INTO TB_FILMS(name, genre, release_year) VALUES('Inception', 'thriller', 2010);
---INSERT INTO TB_FILMS(name, genre, release_year) VALUES('Forrest Gump', 'drama', 1992);
---INSERT INTO TB_FILMS(name, genre, release_year) VALUES('Enter the Dragon', 'adventure', 1973);
---INSERT INTO TB_FILMS(name, genre, release_year) VALUES('Enter the Dragon 2', 'adventure', 1974);
---INSERT INTO TB_FILMS(name, genre, release_year) VALUES('Enter the Dragon 3', 'adventure', 1977);
+INSERT INTO TB_FILMS(name, genre, release_year) VALUES('inception', 'thriller', 2010);
+INSERT INTO TB_FILMS(name, genre, release_year) VALUES('forrest gump', 'drama', 1992);
+INSERT INTO TB_FILMS(name, genre, release_year) VALUES('enter the dragon', 'adventure', 1973);
+INSERT INTO TB_FILMS(name, genre, release_year) VALUES('enter the dragon 2', 'adventure', 1974);
+INSERT INTO TB_FILMS(name, genre, release_year) VALUES('enter the dragon 3', 'adventure', 1977);

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ public class FilmsDTO implements Serializable {`
`23`	`23`
`24`	`24`	`@ApiModelProperty(value = "Field is string")`
`25`	`25`	`@NotBlank(message = "Field name can not be blank")`
`26`		`- @Length(min = 3,max = 50,message = "The field name must have between 3 and 50 character ")`
	`26`	`+ @Length(min = 3, max = 50, message = "The field name must have between 3 and 50 character ")`
`27`	`27`	`private String name;`
`28`	`28`
`29`	`29`	`@ApiModelProperty(value = "Field is string")`
`@@ -51,7 +51,7 @@ public String getName() {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`public void setName(String name) {`
`54`		`- this.name = name;`
	`54`	`+ this.name = name.toLowerCase();`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`public String getGenre() {`
`@@ -69,10 +69,11 @@ public int getYear() {`
`69`	`69`	`public void setYear(int year) {`
`70`	`70`	`this.year = year;`
`71`	`71`	`}`
	`72`	`+`
`72`	`73`	`@Override`
`73`	`74`	`public String toString() {`
`74`	`75`	`return "FilmsDTO [id=" + ", nome=" + name + ", genre=" + genre`
`75`		`- + ", year=" + year + "]";`
	`76`	`+ + ", year=" + year + "]";`
`76`	`77`	`}`
`77`	`78`
`78`	`79`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,8 @@ public List<DetailsFilmsDTOResponse> getAllFilms() {`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`public List<DetailsFilmsDTOResponse> getFilmByName(String nameFilm) {`
`35`		`- films = filmsRepository.findByName(nameFilm);`
	`35`	`+ films = filmsRepository.findByName(nameFilm.toLowerCase());`
	`36`	`+ System.out.println(films);`
`36`	`37`	`if (films.isEmpty()) {`
`37`	`38`	`throw new FilmsDoesNotExistException("There are no films that meet this search");`
`38`	`39`	`}`