🐛 Wrong comparison when comparing submissions within current sem

[sibinhho99]

Cc:

Hi

Describe the Bug

This is a bug of the change introduced in #389 .

When submitting the Arhive.zip file in #399, skeleton code is mysteriously included in current sem submissions' comparisons

This is because there's a careless mistake in the comparison in
https://github.com/WING-NUS/SSID/blob/0fb4da9054d0148605f65cc050a426866e7dfb5a/lib/java/PlagiarismDetection/src/pd/SimComparer.java#L82C5-L82C5 , whereby the s2 submission should be from the currentSemSubmissions variable instead of submissions.

To Reproduce

Steps to reproduce the behavior:

1. Try to submit Archive.zip file in 🐛 Bug when handling skeleton code after implementing references #399
2. Error processing, printing out the processing results shows a "skeleton" student appearing in the comparison results.

Expected Behavior

Correct comparisons made

Tasks

• Investigate
• Fix

Acceptance Criteria

More Information

Environment

• Device: [e.g. iPhone 12]
• Browser: [e.g. chrome, safari]
• OS: [e.g. iOS]

Screenshots

Thanks!