You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
FIXED 1. By default, tumblrgallery extractor add id field to metadata and default filename template has this id field in it.
But there are 2 (that I notice) situation when id field get wrong value.
Examples
Here is 4 json metadata output, first 2 correct and 2 other are wrong:
Correct:
Also, sometimes content\image in gallery have description and link to post. Would be nice to have in metadata description, post id, content\image position number in post (for now there is num property which hold content position at first url, so for gallery link it's content position in gallery and when download post it's position in post) and previous blog name where picture reblogged from when it's available .
For example: https://tumblrgallery.xyz/tumblrblog/gallery/50312.html
First one image has this corwinprescott:“Pulp”Seattle, Wa 2017Corwin Prescott - Gothlet... in description and also link to this post https://tumblrgallery.xyz/post/49178.html, where image is first of six in post.
Or here https://tumblrgallery.xyz/tumblrblog/gallery/50312/7.html ; 2-9 images are in one post with description.
Both posts refers to another blog, so don't really know which blog name should we put in title metadata field. For now title has blog name from https://tumblrgallery.xyz/tumblrblog/gallery/<gallery_id> (so submissive-monarch in this case). Maybe can add one more field for "original" blog name where picture reblogged from.
gallery-dl already support downloading posts. I propose to fix title field, because now it has title of page, and usually (when it has " - " split string and below "see more : ") it consists from blog name and description, so maybe they should be split to title (blog name) and description. If it doesn't have " - " like here, then leave it without split.
Also technically gallery_id contains post id, not gallery.
FIXED 1. By default, tumblrgallery extractor add
id
field to metadata and default filename template has this id field in it.But there are 2 (that I notice) situation when
id
field get wrong value.Examples
Here is 4 json metadata output, first 2 correct and 2 other are wrong:
Correct:
Wrong:
Also, sometimes content\image in gallery have description and link to post. Would be nice to have in metadata description, post id, content\image position number in post (for now there is
num
property which hold content position at first url, so for gallery link it's content position in gallery and when download post it's position in post) and previous blog name where picture reblogged from when it's available .For example:
https://tumblrgallery.xyz/tumblrblog/gallery/50312.html
First one image has this
corwinprescott:“Pulp”Seattle, Wa 2017Corwin Prescott - Gothlet...
in description and also link to this post https://tumblrgallery.xyz/post/49178.html, where image is first of six in post.Or here https://tumblrgallery.xyz/tumblrblog/gallery/50312/7.html ; 2-9 images are in one post with description.
Both posts refers to another blog, so don't really know which blog name should we put in
title
metadata field. For now title has blog name from https://tumblrgallery.xyz/tumblrblog/gallery/<gallery_id> (sosubmissive-monarch
in this case). Maybe can add one more field for "original" blog name where picture reblogged from.gallery-dl already support downloading posts. I propose to fix title field, because now it has title of page, and usually (when it has " - " split string and below "see more : ") it consists from blog name and description, so maybe they should be split to title (blog name) and description. If it doesn't have " - " like here, then leave it without split.
Also technically
gallery_id
contains post id, not gallery.Primary blog name or url. Like in https://tumblrgallery.xyz/tumblrblog/gallery/50312.html this gallery primary blog was submissive-monarch.tumblr.com , so could add url or just submissive-monarch
The text was updated successfully, but these errors were encountered: