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.

Sign up for GitHub

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

Jump to bottom

Issue #1: Technical Debt - Issue in src/.../stats/TrackStatistics.java about overriding hashCode() method #16

Closed
SonamChugh13 opened this issue Feb 14, 2024 · 0 comments · Fixed by #15
Closed

Issue #1: Technical Debt - Issue in src/.../stats/TrackStatistics.java about overriding hashCode() method #16

SonamChugh13 opened this issue Feb 14, 2024 · 0 comments · Fixed by #15
Labels
Group 3 Technical Debt Technical Debt
Milestone
Case Study 1

Comments

@SonamChugh13
Copy link
Owner

Screenshot 2024-02-10 at 11 40 40

Description:
Classes overriding equals() method must also override hashCode() method in order for the class functionality to work correctly. In the TrackStatistics.java class, equals() method has been defined while hashCode() is not.

Location of the issue: TrackStatistics.java
File path: src/main/java/de/dennisguse/opentracks/stats/TrackStatistics.java

Reason for change: To make sure, class functionality works correctly when containers like map are used.
@SonamChugh13 SonamChugh13 added this to the Case Study 1 milestone Feb 14, 2024
@SonamChugh13 SonamChugh13 linked a pull request Feb 14, 2024 that will close this issue
Add hashCode() method to fix issue # 1 #15
Merged
SonamChugh13 pushed a commit that referenced this issue Mar 14, 2024
@beingtranquil
Revert "updated the file KMLTrackExporter" (#16)
b6307d8 
Reverts rilling#14
SonamChugh13 pushed a commit that referenced this issue Mar 14, 2024
@OmniaAlam
Merge pull request #16 from OmniaAlam/dev/Rakibul
5bb94f6 
#14 Refactored updateRecordingDataWhileRecording method for encapsula…
SonamChugh13 pushed a commit that referenced this issue Mar 14, 2024
@WaseemJan063
Merge pull request #16 from tarekFerdous/branch_WJ
6ea936b 
removed the deprecated methods
SonamChugh13 pushed a commit that referenced this issue Mar 14, 2024
@mkjodhani @Avilad0
Fix PMD Violation in Default Value Initializers
e1b830f 
Closes #16

This commit resolves the OMD violation and brings the codebase in line with the established coding standards.
@SonamChugh13 SonamChugh13 mentioned this issue Apr 26, 2024
Merged
emeka-okechukwu pushed a commit that referenced this issue Apr 27, 2024
@chauhan9674
Merge pull request #16 from Vaibhav2399/Inderjeet
c5653e2 
Making the class as Final to avoid exception
emeka-okechukwu pushed a commit that referenced this issue Apr 27, 2024
@VishalPeruma1
Merge pull request #16 from Shrutipavasiya/dev/vishal
ba52958 
Seasons Favorite lifts and merge conflicts fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Group 3 Technical Debt Technical Debt
Projects
None yet
Milestone
Case Study 1
Development

Successfully merging a pull request may close this issue.

Add hashCode() method to fix issue # 1 SonamChugh13/OpenTracks-Group3-SOEN-6431_2024
1 participant
@SonamChugh13