remove movie from library functionality

Author: Ian Redpath

public/project/components/users/user.ts

@@ -50,6 +50,8 @@ export class User {
 				.subscribe(resp => {
 					if (resp.json().library) {
 						const id = resp.json().library.id
+						this.user.libraries.push(resp.json().library)
+						this.userService.login(this.user)
 						this.router.navigate(["/Library", { library: id }])
 					} else {
 						alert(`error creating library ${this.newLibraryName}`)
@@ -58,13 +60,7 @@ export class User {
 		} else {
 			alert("Make sure to name your library first!")
 		}
-/*
-		let newLib: LibraryModel = LibraryModel.defaultLibrary(this.user)
-		newLib.id = new Date().getTime()
-		newLib.name = this.getNextLibraryName()
-		this.userService.addLibrary(newLib)
-		this.router.navigate(["/Library", { library: newLib.id }])
-	*/
+
 	}
 
 	removeLibrary(id: number) {
@@ -74,16 +70,6 @@ export class User {
 					_.remove(this.user.libraries, lib => { return (<any>lib).id === id })
 				}
 			})
-			/*
-		_.remove(this.user.libraries, lib => { return lib.id === id })
-		this.libraryService.removeLibrary(id)
-	*/
-	}
-	getMovies() {
-		/*this.movieService.getMoviesForUser(this.user.id)
-			.subscribe(resp => {
 
-			})
-		*///return _.uniq(_.flatten(this.user.libraries))
 	}
 }

public/project/components/users/user.view.html

@@ -26,8 +26,4 @@ <h3>Libraries</h3>
       </div>
     </div>
   </div>
-  <div>
-    <h3>Highest Rated Movies</h3>
-    <div class="movies"></div>
-  </div>
 </div>

server/project/data/database.ts

@@ -79,6 +79,7 @@ export default class Database {
 
 	createLibrary(library) {
 		library.id = this.randomId++
+		library.movies = []
 		this.libraries.push(library)
 		const user = library.user
 		console.log(user)
@@ -154,25 +155,29 @@ export default class Database {
 		if (_.find(movie.libraries, lib => { return (<any>lib).id === libraryId })) {
 			return movie
 		}
+		const library = _.find(this.libraries, lib => { return lib.id.toString() === libraryId })
+		library.movies.push(movie)
 		let retVal
 		_.forEach(this.movies, mov => {
 			if (mov.id === movie.id) {
-				mov.libraries.push(libraryId)
+				mov.libraries.push(library)
 				retVal = mov
 			}
 		})
 		return retVal
 	}
 
 	deleteMovie(id: string) {
-		const removed = _.find(this.movies, movie => { return movie.id === id })
-		_.remove(this.movies, movie => { return movie.id === id })
+		const removed = _.find(this.movies, movie => { return movie.imdbId === id })
+		_.remove(this.movies, movie => { return movie.imdbId === id })
 		return removed
 	}
 
 	deleteMovieFromLibrary(movieId: string, libraryId: string) {
-		let library = _.find(this.libraries, lib => { return lib.id === libraryId })
-		_.remove(library.movies, mov => { return (<any>mov).id === movieId })
+		let library = _.find(this.libraries, lib => { return lib.id.toString() === libraryId })
+		_.remove(library.movies, mov => { return (<any>mov).imdbId === movieId })
+		let movie = _.find(this.movies, mov => { return mov.imdbId === movieId })
+		_.remove(movie.libraries, lib => { return (<any>lib).id.toString() === libraryId })
 		return library
 	}
 }