Apps/ImageApi
1
0
Fork 0
Code Issues Pull Requests 1 Releases Wiki Activity

Merge pull request 'Just look for date format instead of screenshot text' (#40) from feature/fix-screenshot-date-regex into master

Browse Source 
Reviewed-on: #40
This commit was merged in pull request #40.
This commit is contained in:
cameron
2025-09-01 15:15:49 +00:00
parent ed9071d41d 2cc4124544
commit 5021b3218d
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/memories.rs
View File

@@ -136,7 +136,7 @@ fn extract_date_from_filename(filename: &str) -> Option<DateTime<FixedOffset>> {
// 1. Screenshot format: Screenshot_2014-06-01-20-44-50.png
if let Some(captures) =
regex::Regex::new(r"Screenshot_(\d{4})-(\d{2})-(\d{2})-(\d{2})-(\d{2})-(\d{2})")
regex::Regex::new(r"(\d{4})-(\d{2})-(\d{2})-(\d{2})-(\d{2})-(\d{2})")
.ok()?
.captures(filename)
.and_then(|c| build_date_from_ymd_capture(&c))
@@ -146,7 +146,7 @@ fn extract_date_from_filename(filename: &str) -> Option<DateTime<FixedOffset>> {
// Screenshot format: Screenshot_20140601[_-]204450.png
if let Some(captures) =
regex::Regex::new(r"Screenshot_(\d{4})(\d{2})(\d{2})[_-](\d{2})(\d{2})(\d{2})")
regex::Regex::new(r"(\d{4})(\d{2})(\d{2})[_-](\d{2})(\d{2})(\d{2})")
.ok()?
.captures(filename)
.and_then(|c| build_date_from_ymd_capture(&c))
@@ -428,7 +428,7 @@ mod tests {
#[test]
fn test_extract_date_from_filename_screenshot_underscores_format() {
let filename = "Screenshot_20140601_204450.png";
let filename = "20140601_204450.png";
let date_time = extract_date_from_filename(filename).unwrap();
assert_eq!(date_time.year(), 2014);