RSS-Bridge / rss-bridge

The RSS feed for websites missing it
https://rss-bridge.org/bridge01/
The Unlicense
6.98k stars 1.02k forks source link

Fix: Ensure `$time` is set from `innertext` when `datetime` attribute is not found #4111

Closed Albatrosicks closed 1 month ago

Albatrosicks commented 1 month ago

This commit addresses a bug where the $time variable is not set from the innertext of the $time_element when the datetime attribute is not found. The previous implementation only checked if $time was null or an empty string, which did not cover all cases where the datetime attribute might be missing. By using the empty() function, we ensure that $time is correctly set from the innertext when the datetime attribute is not present.

github-actions[bot] commented 1 month ago

Pull request artifacts

Bridge Context Status
CssSelectorComplex 1 untitled (current) Bridge returned error 404! (19862)
Type: HttpException
Message: https://example.com/blog/ resulted in 404 Not Found <!doctype html>
CssSelectorComplex 1 untitled (pr) Bridge returned error 404! (19862)
Type: HttpException
Message: https://example.com/blog/ resulted in 404 Not Found <!doctype html>

last change: Sunday 2024-05-19 08:43:48

dvikan commented 1 month ago

@LarsStegman