[actual destination: nus-cs2113-ay1920s2/pe-moderation actual assignee: Tejasvas123]
Issue Description
I'm allowed to add in duplicated task with the same name as along they are not exact copies of each other. This will be confusing for the user if he/she added a duplicated task by accident and will not be able to identify the difference. It will be good to make the task name CASE INSENSITIVE.
[original: benchan911/pe#7]
Team's Response
The names of the directories are case-sensitive. This was intended as it simulates how file names are also case-sensitive in some file systems, and LINUX also has case-sensitve directory names.
Thus, names of directories that are "same" and only differ in cases will be considered as different names. The only exception is module code, since they are unique regardless of cases, and are not a user-made name. This can be shown in the User Guide:
However, we are willing to accept the bug since we may have failed to adequately convey this difference. Thank you for pointing this bug out.
To Check [3 Items]
:question: Issue type
Team chose [type.DocumentationBug]
Originally [type.FunctionalityBug]
Tester's objection: It is explained that module code is case-insensitive. However, the problem that was raised in this issue is regarding the adding of task.
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Medium]
Tester's objection: I would think that the issue severity would be low instead of VeryLow because it does cause a bit of a hindrance when learning how to use Nuke and the differences between the case-sensitive and case-insensitive.
:question: Duplicate status
The team claimed this to be an duplicate of the following bug:
Title: [DG] DuplicateTaskException
Description:
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
DuplicateTaskException was not thrown when Lab 4 already exists as a task and lab 4 is added.
[actual destination: nus-cs2113-ay1920s2/pe-moderation actual assignee: Tejasvas123]
Issue Description
I'm allowed to add in duplicated task with the same name as along they are not exact copies of each other. This will be confusing for the user if he/she added a duplicated task by accident and will not be able to identify the difference. It will be good to make the task name CASE INSENSITIVE.
[original: benchan911/pe#7]
Team's Response
The names of the directories are case-sensitive. This was intended as it simulates how file names are also case-sensitive in some file systems, and LINUX also has case-sensitve directory names. Thus, names of directories that are "same" and only differ in cases will be considered as different names. The only exception is
module code, since they are unique regardless of cases, and are not a user-made name. This can be shown in the User Guide:However, we are willing to accept the bug since we may have failed to adequately convey this difference. Thank you for pointing this bug out.
To Check [3 Items]
:question: Issue type
Team chose [
type.DocumentationBug] Originally [type.FunctionalityBug]Tester's objection: It is explained that module code is case-insensitive. However, the problem that was raised in this issue is regarding the adding of task.
:question: Issue severity
Team chose [
severity.VeryLow] Originally [severity.Medium]Tester's objection: I would think that the issue severity would be low instead of VeryLow because it does cause a bit of a hindrance when learning how to use Nuke and the differences between the case-sensitive and case-insensitive.
:question: Duplicate status
The team claimed this to be an duplicate of the following bug:
Title: [DG] DuplicateTaskException Description: Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
DuplicateTaskException was not thrown when
Lab 4already exists as a task andlab 4is added.[original: nus-cs2113-AY1920S2/pe-interim#197]