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.

Sign up for GitHub

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

Jump to bottom

Check isNaN on alMax & use today as birthdate max #248

Merged
merged 5 commits into from
Feb 19, 2024
Merged

Check isNaN on alMax & use today as birthdate max #248

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e25dd7b
Check isNaN on alMax & use today as birthdate max
BryceStevenWilley Jan 19, 2024
544caca
Log when an invalid date is seen
BryceStevenWilley Jan 19, 2024
8c94457
Add missing parens in JS code string
BryceStevenWilley Jan 23, 2024
9826c94
Testing changes:
BryceStevenWilley Jan 24, 2024
306bd90
Comment pointing out consistency
BryceStevenWilley Feb 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 26 additions & 16 deletions docassemble/ALToolbox/ThreePartsDate.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -510,14 +510,17 @@
// Non-MVP. Make an issue.
// Replace '-' in case it's an ISO date format, (our recommended format).
// JS doesn't play nicely with ISO format.
var min_attr = get_$original_date(field).attr('data-alMin');
if (!min_attr) {{
// Validation should always succeed if no minimum given
let min_attr = get_$original_date(field).attr('data-alMin') || "";
let min_date = new Date(min_attr.replace(/-/g, '/'));
if (isNaN(min_date)) {{
if (min_attr !== "") {{
console.log(`The alMin attribute (${{ min_attr }}) isn't a valid date!`);
}}
// Validation should always succeed if no or bad minimum given
return true;
}}
var min_date = new Date(min_attr.replace(/-/g, '/'));

// Avoid usinig `params`, which could be in many different formats.
// Avoid using `params`, which could be in many different formats.
// Just get date data from the actual fields
var data = get_date_data(field);
var date_input = new Date(data.year + '/' + data.month + '/' + data.day);
Expand All @@ -526,7 +529,8 @@

}}, function alMinMessage (validity, field) {{
/** Returns the string of the invalidation message. */
let min_date = new Date(get_$original_date(field).attr('data-alMin').replace(/-/g, '/'));
let min_attr = get_$original_date(field).attr('data-alMin') || "";
BryceStevenWilley marked this conversation as resolved.
Show resolved Hide resolved
let min_date = new Date(min_attr.replace(/-/g, '/'));
let locale_long_date = min_date.toLocaleDateString(undefined, {{ day: '2-digit', month: 'long', year: 'numeric' }});
return (
get_$original_date(field).attr('data-alMinMessage')
Expand All @@ -543,17 +547,22 @@
}}

// TODO: Catch invalid alMax attr values for devs? Log in console? Make post MVP issue
let max_attr = get_$original_date(field).attr('data-alMax');
if (!max_attr) {{
// Validation should always succeed if no minimum given
return true;
}}
let max_attr = get_$original_date(field).attr('data-alMax') || "";
let max_date = new Date(max_attr.replace(/-/g, '/'));
if ( isNaN(max_date) && is_birthdate(field)) {{
max_date = new Date();
if (isNaN(max_date)) {{
if (max_attr !== "") {{
console.log(`The alMax attribute (${{ max_attr }}) isn't a valid date!`);
}}
if (!is_birthdate(field)) {{
// Validation should always succeed if no or bad max given on normal dates
return true;
}} else {{
// if birthdate and no or bad max given, assume today is the max date
max_date = new Date();
}}
}}

// Avoid usinig `params`, which could be in many different formats.
// Avoid using `params`, which could be in many different formats.
// Just get date data from the actual fields
var data = get_date_data(field);
var date_input = new Date(data.year + '/' + data.month + '/' + data.day);
Expand All @@ -562,11 +571,12 @@

}}, function alMaxMessage (validity, field) {{
/** Returns the string of the invalidation message. */
let max_date = new Date(get_$original_date(field).attr('data-alMax').replace(/-/g, '/'));
let max_attr = get_$original_date(field).attr('data-alMax') || "";
plocket marked this conversation as resolved.
Show resolved Hide resolved
let max_date = new Date(max_attr.replace(/-/g, '/'));
let locale_long_datetime = max_date.toLocaleDateString(undefined, {{ day: '2-digit', month: 'long', year: 'numeric' }})
let default_MaxMessage = `The date needs to be on or before ${{ locale_long_datetime }}.`;
// Birthdays have a different default max message
if (!get_$original_date(field).attr('data-alMax') && is_birthdate(field)) {{
if (is_birthdate(field) && isNaN(max_date)) {{
default_MaxMessage = 'A <strong>birthdate</strong> must be in the past.';
}}

Expand Down