NabiKAZ / namava-downloader

Download movies from `namava.ir` website. | لینک جایگزین جهت احتیاط: https://gitlab.com/NabiKAZ/namava-downloader
GNU General Public License v3.0
104 stars 15 forks source link

Switch video input from video id to video url #7

Closed mahdyar closed 6 years ago

mahdyar commented 6 years ago

Now you can send video url , instead of video id !

NabiKAZ commented 6 years ago
ممنونم دوست من. ولی من متوجه نشدم، شما تنها کلمه ID رو به URL تغییر دادید.
mahdyar commented 6 years ago

$video_id = explode("/",$video_id)[4]; خط 51 ، کار سختی نبود :)) بر اساس اسلش تجزیش میکنه و آیدی رو از لینک میکشه بیرون ..

NabiKAZ commented 6 years ago
حق با شما بود و من به خط 51 دقت نکرده بودم. اما آدرس های عمومی برای نمونه: ``` https://www.namava.ir/%D8%A7%D8%AC%D8%AA%D9%85%D8%A7%D8%B9%DB%8C/37/15143 ``` دارای 5 اسلش هستند نه 4 اسلش که در این مورد کد شما باگ داره. همچنین برای استیبل بودن بیشتر به نظرم بر اساس عدد آخر نرسیده به اولین اسلش اگر آیدی خونده بشه، بهتره. از طرفی چون کاربران قبلی به ورود آیدی عادت دارند. میتونه دو حالت همزمان پیاده بشه. یعنی اگر عدد وارد شد مثل رویه قبلی و اگر آدرس وارد شد، از روی اون آیدی گرفته بشه. از توجهتون ممنونم.
mahdyar commented 6 years ago

بله ظاهرا با بی دقتی باگ نوشته بودم :)) خواستم یه کاری کرده باشم خلاصه :+1:

NabiKAZ commented 6 years ago
قطعاً حرکت و نیت همکاری شما ارزشمند بوده. کد هم اگه باگ نداشته باشه که کد نیست 😉 در مجموع اینکه علاوه بر آیدی، آدرس صفحه رو هم بگیره فکر خوبیه و باید بیشتر روش کار بشه. (البته روز اول من مخصوصاً اینکار رو نکردم چون آدرسها ممکنه پارامترهای اضافه داشته باشند و کار تشخیص همه حالتهای ممکنه برای تشخیص آیدی رو سخت میکنه. به همین جهت راه ساده و گرفتن تنها آیدی عددی رو در پیش گرفتم) بازهم اگر نکته ای بود خوشحال میشم مطرح بشه.