Fixed bug196 - Bizarre options on "list courses by faculty" dropdown list #46

Merged
mrlan merged 2 commits from Bug196-Ward into master 2022-06-20 07:03:44 +08:00

Bug Description

When clicking on the "list courses by faculty" dropdown list, you'll find bizarre list items for example:

"Search by faculty" item duplicated five times.

There are items named "1" and "2".

And there is blank items!!!.

Steps to Reproduce:

  1. login onto LRR
  2. Click on "Search courses by faculty"

Fixing Procedures

Fixing this bug actually was easy all I had to do is just move the title of the dropdown menu to outside the while loop to stop it from duplicating.

### Bug Description When clicking on the "list courses by faculty" dropdown list, you'll find bizarre list items for example: "Search by faculty" item duplicated five times. There are items named "1" and "2". And there is blank items!!!. Steps to Reproduce: 1) login onto LRR 2) Click on "Search courses by faculty" ### Fixing Procedures Fixing this bug actually was easy all I had to do is just move the title of the dropdown menu to outside the while loop to stop it from duplicating.
3279308836 self-assigned this 2022-06-19 20:52:37 +08:00
3279308836 added 1 commit 2022-06-19 20:52:37 +08:00

@3279308836

Thanks. I found a regression error. Please check your email.

Hui

@3279308836 Thanks. I found a regression error. Please check your email. Hui
3279308836 added 1 commit 2022-06-20 05:15:31 +08:00

Worked. You have fixed Bug 196, and the regression error is gone. Thanks.

-Hui

Worked. You have fixed [Bug 196](http://118.25.96.118/bugzilla/show_bug.cgi?id=196), and the regression error is gone. Thanks. -Hui
mrlan merged commit 6831fdcd0f into master 2022-06-20 07:03:44 +08:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mrlan/LRR#46
There is no content yet.