| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
iterate_over_comments() is a sub to handle all comments.
print_comment_chain() can only parse first kind of comments now, i.e.
top-level comments & their replies.
Comments hidden under "load more comments" & "continue this thread"
require other operations before print_comment_chain() can print them.
All those operations are handled by iterate_over_comments().
Currently "continue this thread" is skipped but "load more comments"
is parsed. This doesn't add anything to previous releases but just
changes the structure.
I'm just preparing to add code to parse comments hidden under
"continue this thread". After that Draco will be able to archive the
whole thread. Previously handling this would've required copying the
same code over 3 times, now it'll be handled at one place.
3 times would've been:
1. when we start iterating over top-level comments.
2. when comments hidden under "load more comments" are parsed &
their child is hidden under "continue this thread".
3. when replies of normal comments are hidden under "continue this
thread".
Case 1 & 2 is fixed by switching to iterate_over_comments(). For
third, the replies are handled by iterate_over_comments() so the 3rd
case is also fixed.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
$comment in print_comment_chain() was changed to $comment_data to
prevent confusion in future.
We have to check if url has "/" at the end because if it does have it
then previously we were appending "$comment_data->{id}.json" to it
which makes the url look like:
http://<reddit>/<post_id>//<comment_id>.json
And reddit doesn't like that, reddit wants it to be:
http://<reddit>/<post_id>/<comment_id>.json
Notice the double '/' in previous link.
|
| |
|
|
|
|
|
|
|
| |
This version makes the script lot more complex.
This version will load all those comments hidden behind "load more
comments". But not those hidden behind "continue this thread".
Starting this release we have a News section.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
Unicode::LineBreak seems better.
|
| |
|
|
|
|
| |
Space before each line was done because some text may contain `*' at
the start of line which confuses Org & it presents that as a new
heading. Inserting a space before every new line fixes this.
|
| |
|