ankitsejwal / Lyndor

:rocket: Powerful command line tool to download lynda.com courses for personal offline use. :part_alternation_mark:
MIT License
131 stars 32 forks source link

TypeError: can only concatenate str (not "NoneType") to str #50

Closed jawidjrs closed 6 years ago

jawidjrs commented 6 years ago

Please follow the guide below


Make sure you are using the latest version: run git pull to update your version from Lyndor directory

Before submitting an issue make sure you have:

What is the purpose of your issue?


If the purpose of this issue is a bug report, or you are not completely sure then provide the full terminal output as follows:

Copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

Microsoft Windows [Version 10.0.17134.228]
(c) 2018 Microsoft Corporation. All rights reserved.

C:\Users\Jawid Payanda>cd C:\Users\Jawid Payanda\Videos\Lynda

C:\Users\Jawid Payanda\Videos\Lynda>Run-Lyndor.bat
             .
           .d$ .
           d$$  :
          .$$$
          :$$$   :            Download multiple courses
          $$$$   :          at a time, just paste all the
          $$$$   :            urls in Bulk Download.txt
         .$$$$
         :$$$$    :
         $$$$$    :
         :    $$$$$      Change preferences in settings.json
         :    $$$$$        Run python settings/settings.py
        .:    $$$$$.
       / :    $$$$: \
      /  :    $$$$:  \
     '        $$$$`   '
     |   /:   $$$$\   |
     |  /     $$$` \  |
     |_/   :__$$P   \_|    🏄

1. Paste course url or
2. Press enter for Bulk Download
https://www.lynda.com/SQL-tutorials/Learning-SQL-Programming/548044-2.html

⛺  Using username and password combination for download

Course Name     Learning SQL Programming
Course id       548044
Author Name     Scott Simpson
Topics          #Developer      #Databases
Softwares       #SQL    #SQLite
Duration        1h 27m
Release Date    8/15/2017
Downloaded On   24/August/2018
Course URL      https://www.lynda.com/SQL-tutorials/Learning-SQL-Programming/548044-2.html

🌟  info.txt created

Creating Chapters:

00. Introduction
01. Ask for Data from a Database
02. Ask for Data from Two or More Tables
03. Data Types  Math  and Helpful Functions
04. Add or Modify Data
05. Conclusion

✅  6 chapters created!!

👍🏻  CONTENT.md created.

Traceback (most recent call last):
  File "C:\Users\Jawid Payanda\Downloads\Programs\Lyndor-master/run.py", line 133, in <module>
    main()
  File "C:\Users\Jawid Payanda\Downloads\Programs\Lyndor-master/run.py", line 35, in main
    schedule_download(url)
  File "C:\Users\Jawid Payanda\Downloads\Programs\Lyndor-master/run.py", line 51, in schedule_download
    download_course(url)
  File "C:\Users\Jawid Payanda\Downloads\Programs\Lyndor-master/run.py", line 101, in download_course
    save.videos(url, cookie_path, course_folder_path)   # Download videos
  File "C:\Users\Jawid Payanda\Downloads\Programs\Lyndor-master\save.py", line 234, in videos
    cookie = ' --cookies ' + '"' + cookie_path + '"'     # cookie
TypeError: can only concatenate str (not "NoneType") to str
Press any key to continue . . .

Answer questions related to your Environment which will help in reproducing the issue:

The issue was encountered on: :computer:

Enter the python version you are using for download. Find your python version by typing in terminal python -V


If the purpose of this issue is a bug report please provide all kinds of example URLs where you encountered issues (replace following example URLs by yours):

https://www.lynda.com/MySQL-tutorials/Database-Clinic-MySQL/609013-2.html https://www.lynda.com/SQLite-tutorials/Database-Clinic-SQLite/588032-2.html

Description of your issue, suggested a solution and other information

Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible. Every time I run run-lyndor.bat on command prompt it encounters with same error.

ankitsejwal commented 6 years ago

Hi @jawidjrs sorry for messing things up with the last update, last update was relatively big and had some breaking changes hence you are facing these issues. Please do report if you find any more once you test this patch. Thanks.