Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FR]: iterate current date for articles with no own date #621

Closed
Owyn opened this issue Feb 4, 2022 · 6 comments
Closed

[FR]: iterate current date for articles with no own date #621

Owyn opened this issue Feb 4, 2022 · 6 comments
Assignees
Labels
Component-Message-List Status-Fixed Ticket is resolved. Type-Enhancement This is request for brand new feature.
Milestone

Comments

@Owyn
Copy link
Contributor

Owyn commented Feb 4, 2022

Brief description of the feature request

new_message.m_created = current_time;

currently it sets same current time for all articles from a feed without their own published-date specified

FR: - subtract article index (1st arcticle, 2nd article... 99th article) from the date-time so it would become possible to sort by time in the RSSGuard for articles without specified pubDate in their feed (pretty much everything generated via script - cuz on sites ppl don't specify publish-date of grabbed-parsed links),
so articles at the top of the feed would get placed at the top of the rssguard articles list window when sorted by date

e.g.:
Title | Date
Most Recent article - 15:05
Yesterdays article - 15:04
Very old article- 15:03

how it is now:
Title | Date
Very old article - 15:05
Most Recent article - 15:05
Yesterdays article - 15:05

@Owyn Owyn added the Type-Enhancement This is request for brand new feature. label Feb 4, 2022
@martinrotter
Copy link
Owner

URL of some feed to test this?

@martinrotter
Copy link
Owner

Nevermind, fixed. I set the increment to 1 second.

image

@martinrotter martinrotter added this to the 4.2.0 milestone Feb 5, 2022
@Owyn
Copy link
Contributor Author

Owyn commented Feb 5, 2022

Thx, this'd be a great improvement for when you add a (pubDate-less) feed for the first time, before everything had the same time and couldn't be sorted by date

@Owyn
Copy link
Contributor Author

Owyn commented Feb 5, 2022

Is the needed commit not out yet?

I tried dev
Version: 4.2.0 (built on Windows/AMD64) Revision: 5ef90cc6-nowebengine
with related commits above and the time is still all the same

image

url: https://asurascans.com
script: python css2rss.py "div.luf" "h4" "li > a" "li > a" "li > a"

@martinrotter
Copy link
Owner

Yes, I completely forgot about JSON feeds. Will fix.

@martinrotter
Copy link
Owner

fixed for json too

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component-Message-List Status-Fixed Ticket is resolved. Type-Enhancement This is request for brand new feature.
Projects
None yet
Development

No branches or pull requests

2 participants