search

popcorn-official / popcorn-api

Popcorn Time is a multi-platform, free software BitTorrent client that includes an integrated media player. Compatible API Anime/Movies/Show Scrapper
https://popcorntime.app
MIT License
622 stars 101 forks source link

CLI error when adding a movie. #43

Closed jhches21 closed 8 years ago

jhches21 commented 8 years ago

Did gulp build of the latest master (v2.2.0) and tried to invoke the CLI by -

./popcorn-api.js -c movie

After inputting all of information, the error pops up -

An error occurred: ReferenceError: logger is not defined

Backtracking through all codes, I figured it was due to the logger not being referenced in moviehelper.js Although I'm not too sure how I'm supposed to reference logger to something.

ChrisAlderson commented 8 years ago

You're most likely right. The CLI part of the API is the least robust part of it, but I'm already busy with some smaller patches for v2.2.1 so I'll try to include fixes for your issues.

jhches21 commented 8 years ago

Okay, I'll more than happy to try and fix the rest of issue with CLI if I can find them.

As long I can fix them and still follow this project style guideline, I'll submit pull request.

ChrisAlderson commented 8 years ago

Of course you are also welcome to do that.

ChrisAlderson commented 8 years ago

This should be fixed in v2.2.1 (5373d4e45432a66aa975ccc0b11e419639e2add3) so I'm closing the issue, if I'm wrong feel free to re-open the issue.