Hours of Operation using Matrix - expressionengine

Question

I am building a directory of businesses and want to not only publish a list of business hours, but also to publish if a business is currently open for business.
In a matrix, I have 7 rows with row_1 representing Sunday row_7 for Saturday. So I have two questions.
Is this as concise as the code COULD be or is there a better way?
Is there a flaw in the conditional that tells whether a business is currently open? It seems to work right now, but not very tested.
{!-- Hours of Operation --}
{exp:stash:set name="hours-of-operation"}
The Current time is: {current_time format="%g:%i%a"}<br/>
{hours_of_operation}
{if row_count=="1"}Sunday{/if}
{if row_count=="2"}Monday{/if}
{if row_count=="3"}Tuesday{/if}
{if row_count=="4"}Wednesday{/if}
{if row_count=="5"}Thursday{/if}
{if row_count=="6"}Friday{/if}
{if row_count=="7"}Saturday{/if}
{open_time format="%g:%i%a"} - {close_time format="%g:%i%a"}<br/>
{/hours_of_operation}
{/exp:stash:set}
{!-- Hours of Operation --}
{!-- Are we open? --}
{exp:stash:set name="are-we-open"}
{exp:mx_calc expression='{current_time format="%w"}+1'}
{!-- matrix --}
{hours_of_operation}
{if row_count=="{calc_result}"}
Today is: {current_time format="%l"}<br/>
<strong>
{if '{open_time format="%H%i"}' <= '{current_time format="%H%i"}' && '{close_time format="%H%i"}' <= '{current_time format="%H%i"}'}
We are currently open!{if:else}We are currently closed.
{/if}
</strong><br/>
Today's Hours are:<br/> <strong>{open_time format="%g:%i%a"} - {close_time format="%g:%i%a"}</strong><br/>
{/if}
{/hours_of_operation}
{!-- matrix --}
{/exp:mx_calc}
{/exp:stash:set}
{!-- Are we open? --}

Answer 1

This looks good to me, the only thing I would change is add another column on the left of the matrix and call it day of week with a drop down to allow the client to select the day. then in your code you can get rid of all those conditionals and replace it with just {day_of_week}

Answer 2

This logic shouldn't work:
{if '{open_time format="%H%i"}' <= '{current_time format="%H%i"}' && '{close_time format="%H%i"}' <= '{current_time format="%H%i"}'}
You're checking that both the closing and opening times are less than current_time, rather than checking that current_time is between the two values. If the business is open then close_time should be more than current_time, not less. The logic should be:
{if
'{open_time format="%H%i"}' <= '{current_time format="%H%i"}' &&
'{close_time format="%H%i"}' > '{current_time format="%H%i"}'
}
Also if we're being picky, what do people do if they have to input data for a business that's closed entirely for one or more days of the week? If it were me I'd throw a PT Switch field in as a 'Closed All Day' column, defaulting to no. It would only require a small tweak to your existing logic:
{if
'{open_time format="%H%i"}' <= '{current_time format="%H%i"}' &&
'{close_time format="%H%i"}' > '{current_time format="%H%i"}' &&
'{closed_all_day}' != 'y'
}
We're currently open!
{if:else}
Then in the {hours_of_operation} loop:
{if closed_all_day != 'y'}
{open_time format="%g:%i%a"} - {close_time format="%g:%i%a"}<br/>
{else}
Closed<br/>
{/if}