// This would be better implemented if weeknumber wasn't an argument, but
        // rather it loaded according to currentWeekNumber.  I can't imagine a
        // scenario in which you'd want to load something different from
        // currentWeekNumber.  This was what was causing the previous bug with
        // edits in one week showing up in another. ?? Did I write this C.G?
        // Cause I'm thinking selecting another week number in the Combobox means that
        // you can jump around week numbers
        private void LoadWeek(int weeknumber)
        {
            isInitialising = true;

            //Clear the current rows
            dgScheduleView.Rows.Clear();
            try
            {
                IList <ScheduleRecord> list = scheduleModel.GetWeek(weeknumber);

                foreach (ScheduleRecord record in list)
                {
                    //match the fields in record with those to go in the datagrid
                    dgScheduleView.Rows.Add(CreateDataGridRow(record));
                }

                lblTitle.Text = ScheduleRecord.WeekName(weeknumber - 1);
            }
            catch (Exception e)
            {
                MessageBox.Show("Exception occured loading week " + weeknumber + "\r\n" + e.ToString(), "Exception Loading Week", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
            finally
            {
                isInitialising = false;
            }
        }