Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add command to view details of an item #44

Merged
merged 4 commits into from
Oct 8, 2020

Conversation

justacasul
Copy link

also changed the UG link

@justacasul justacasul added the feature.Item Issues related to items. label Oct 6, 2020
change gui name
@codecov-io
Copy link

Codecov Report

Merging #44 into master will decrease coverage by 0.65%.
The diff coverage is 42.02%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #44      +/-   ##
============================================
- Coverage     63.21%   62.55%   -0.66%     
- Complexity      730      747      +17     
============================================
  Files           124      128       +4     
  Lines          2539     2604      +65     
  Branches        269      278       +9     
============================================
+ Hits           1605     1629      +24     
- Misses          847      882      +35     
- Partials         87       93       +6     
Impacted Files Coverage Δ Complexity Δ
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...va/seedu/address/logic/parser/InventoryParser.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
src/main/java/seedu/address/ui/HelpWindow.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...n/java/seedu/address/ui/InventoryDetailedCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...main/java/seedu/address/ui/InventoryListPanel.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...ain/java/seedu/address/ui/InventoryMainWindow.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...edu/address/model/item/NameIsExactlyPredicate.java 60.00% <60.00%> (ø) 7.00 <7.00> (?)
...edu/address/logic/commands/ViewDetailsCommand.java 72.72% <72.72%> (ø) 6.00 <6.00> (?)
...va/seedu/address/logic/commands/CommandResult.java 78.94% <75.00%> (-9.29%) 10.00 <2.00> (ø)
...in/java/seedu/address/commons/util/StringUtil.java 91.66% <80.00%> (-3.08%) 8.00 <1.00> (+1.00) ⬇️
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc9e99c...8a6afa0. Read the comment docs.

@justacasul justacasul merged commit 7e4760e into AY2021S1-CS2103T-F13-1:master Oct 8, 2020
Comment on lines +52 to +60
public static boolean containsMultipleWordsIgnoreCase(String sentence, String words) {
requireNonNull(sentence);
requireNonNull(words);

String preppedWord = words.trim();
checkArgument(!preppedWord.isEmpty(), "Word parameter cannot be empty");

return sentence.equalsIgnoreCase(preppedWord);
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be better to name it as "exactlyMultipleWordsIgnoreCase"? "contains" is a little misleading.

Copy link

@Rahul0506 Rahul0506 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change in method name, other than that LGTM

@justacasul justacasul added this to the v1.2 milestone Oct 16, 2020
@justacasul justacasul self-assigned this Oct 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature.Item Issues related to items.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update help link As a user, I want to be able to view detailed information about an item
5 participants