deniscerri / ytdlnis

Android Video/Audio Downloader app using yt-dlp
GNU General Public License v3.0
3.32k stars 117 forks source link

[BUG] Video downloads music file instead when using M4A music format #453

Closed fg817sw751478 closed 3 months ago

fg817sw751478 commented 3 months ago

** Before making an issue make sure you are using the latest version of yt-dlp in the settings. Perhaps try using the nightly version of yt-dlp. If that didn't help, proceed with the issue

Describe the bug A clear and concise description of what the bug is. When using M4A as a music format in videos, for some reason, the downloader only downloads a music file.

To Reproduce Steps to reproduce the behavior:

  1. Download a video with M4A music format
  2. The music file gets downloaded instead (and is in the videos folder too)

Expected behavior A clear and concise description of what you expected to happen. My video would download normally.

Screenshots / Video Recording If applicable, add them to help explain your problem. None

(please complete the following information):

Additional context Add any other context about the problem here. When downloading with Opus, issue is fixed.

tusharchopra07 commented 3 months ago

Show your extra commands screenshot or just paste it , or if you can provide log of the download

fg817sw751478 commented 3 months ago

Show your extra commands screenshot or just paste it , or if you can provide log of the download

Here

~ $ yt-dlp -N "3" --trim-filenames "183" --no-mtime --parse-metadata "uploader:(?P<uploader>.+)(?: - Topic)$" --parse-metadata "%(uploader,channel,creator,artist|null)s:%(uploader)s" --embed-chapters -S "+hasaud,vcodec:h264,acodec:aac" -f "136+140-1/140/136+ba/136/b" --embed-subs --sub-langs "en.*,.*-orig" -P "/storage/emulated/0/Android/data/com.deniscerri.ytdl/files/downloads/66" -o "%(uploader)s - %(title)s.%(ext)s" --replace-in-metadata "video:title" ".+" "\"Ridiculous\" - DeMarcus Cousins" --replace-in-metadata "video:uploader" ".+" "Cee Dee" "https://youtube.com/watch?v=xuoNUQ5s2Z4"

[youtube] Extracting URL: https://youtube.com/watch?v=xuoNUQ5s2Z4
[youtube] xuoNUQ5s2Z4: Downloading webpage
[youtube] xuoNUQ5s2Z4: Downloading ios player API JSON
[youtube] xuoNUQ5s2Z4: Downloading android player API JSON
[youtube] xuoNUQ5s2Z4: Downloading m3u8 information
[MetadataParser] Could not interpret 'uploader' as '(?P<uploader>.+)(?: - Topic)$'
[MetadataParser] Parsed uploader from '%(uploader,channel,creator,artist|null)s': 'Cee Dee'
[info] xuoNUQ5s2Z4: Downloading 1 format(s): 140
[MetadataParser] Changed title to: "Ridiculous" - DeMarcus Cousins
[MetadataParser] Changed uploader to: Cee Dee
[info] There are no subtitles for the requested languages

[FixupM4a] Correcting container of "/storage/emulated/0/Android/data/com.deniscerri.ytdl/files/downloads//TERMINAL/4/Cee Dee - "Ridiculous" - DeMarcus Cousins.m4a"
[EmbedSubtitle] There aren't any subtitles to embed
[Metadata] There isn't any metadata to add
[download] 100% of  377.60KiB in 00:00:00 at 756.89KiB/s 
[FixupM4a] Correcting container of "/storage/emulated/0/Android/data/com.deniscerri.ytdl/files/downloads//TERMINAL/4/Cee Dee - "Ridiculous" - DeMarcus Cousins.m4a"
[EmbedSubtitle] There aren't any subtitles to embed
[Metadata] There isn't any metadata to add
tusharchopra07 commented 3 months ago

Hmm can you share a screen record too (with configurations )

zaednasr commented 3 months ago

Yes this appears to be a bug. Will fix

zaednasr commented 3 months ago

check 1.7.5