Fix: Incorrect SQL statement in update_days_attended of mod.rs

[he1senbrg]

There is issue with the numbering of placeholders in the update_days_attended function.

The UPDATE statement had:

UPDATE AttendanceSummary
SET days_attended = days_attended + 1
WHERE member_id = $2
AND year = $3
AND month = $4

This was wrong because the placeholders didn’t match the parameters being passed. The correct order should be:

UPDATE AttendanceSummary
SET days_attended = days_attended + 1
WHERE member_id = $1
AND year = $2
AND month = $3

Similarly, the below INSERT statement didn't have the placeholder $3 specified for month.

sqlx::query(
                r#"
                    INSERT INTO AttendanceSummary (member_id, year, month, days_attended)
                    VALUES ($1, $2, $3, 1)
                "#,
            )
            .bind(member_id)
            .bind(year)
            .bind(month)
            .execute(pool)
            .await
            .unwrap();

[ivinjabraham]

Crazy how this stayed unnoticed for so long. Good catch, though I would prefer having a description of the fix and/or bug in the commit message so it's easier to review later.

[he1senbrg]

@ivinjabraham I have added the description.

[ivinjabraham]

Heh, I meant in the commit message since that's what we'll be looking at in the future. But don't sweat it, I'll merge this in a while. Just making a few other changes in the meantime.

[he1senbrg]

oops, I'll make sure to word commits better next time.