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

Removing the Deprecated lines in the code Group(09) #15

Closed
WaseemJan063 opened this issue Feb 16, 2024 · 3 comments
Closed

Removing the Deprecated lines in the code Group(09) #15

WaseemJan063 opened this issue Feb 16, 2024 · 3 comments
Labels
Group 09 Group_9_SCM_Project Technical Debt

Comments

@WaseemJan063
Copy link
Collaborator

WaseemJan063 commented Feb 16, 2024 •
edited
Loading

Describe the bug
Code is sometimes annotated as deprecated by developers maintaining libraries or APIs to indicate that the method, class, or other programming element is no longer recommended for use. This is typically due to the introduction of a newer or more effective alternative. For example, when a better solution has been identified, or when the existing code presents potential errors or security risks.

Deprecation is a good practice because it helps to phase out obsolete code in a controlled manner, without breaking existing software that may still depend on it. It is a way to warn other developers not to use the deprecated element in new code, and to replace it in existing code when possible.

Location: src/main/java/de/dennisguse/opentracks/TrackStoppedActivity.java
Screenshot 2024-02-16 at 8 18 18 PM

Proposed Solution:
image

Technical information

  • Device: [e.g. Nexus 7 (2013)]
  • OS: [e.g. LineageOS 15.1]
  • OpenTracks version: [e.g. v3.2.4]
  • OpenTracks commit id (only for nightly builds): [e.g. 4b333bb]
@WaseemJan063
Copy link
Collaborator Author

image

@WaseemJan063
Copy link
Collaborator Author

WaseemJan063 commented Feb 16, 2024 •
edited
Loading

I worked on this issue in the file: TackstoppedActivity.java and resolved the issue

@WaseemJan063 WaseemJan063 changed the title Removing the Deprecated lines in the code Removing the Deprecated lines in the code Group(09) Feb 16, 2024
@Sharanyu Sharanyu added Group 09 Group_9_SCM_Project Technical Debt labels Feb 17, 2024
@WaseemJan063
Copy link
Collaborator Author

image

issue resolved.

@Sharanyu Sharanyu mentioned this issue Feb 17, 2024
Merged
Sharanyu added a commit to rilling/OpenTracks-Winter-SOEN-6431_2024 that referenced this issue Feb 17, 2024
@Sharanyu
Fix: Group9 issues: #1 #2 #6 #9 #11 #13 #14 #15 (#50)
35390eb 
This PR consists of changes for issues resolved by Group9.


[#1](tarekFerdous#1)
Define a constant instead of duplicating this literal "<open>1</open>" 3
times.

[#2](tarekFerdous#2)
Replace this usage of 'Stream.collect(Collectors.toList())' with
'Stream.toList()' Group 09

[#6](tarekFerdous#6)
Either re-interrupt this method or rethrow the "InterruptedException"
that can be caught here.

[#9](tarekFerdous#9)
Remove unused "TAG" private field.

[#11](tarekFerdous#11)
Use isEmpty() to check whether the collection is empty or not.

[#13](tarekFerdous#13)
Fixing technical debts for io/file/impoter/importService.java and
io/file/importer/importViewModel.java

[#14](tarekFerdous#14)
io/file/importer/KmlTrackImporter and io/file/importer/KmzTrackImporter

[#15](tarekFerdous#15)
Removing the Deprecated lines in the code Group(09)
beingtranquil pushed a commit that referenced this issue Mar 17, 2024
@SonamChugh13
Merge pull request #15 from SonamChugh13/sonam
d5c5cf2 
Add hashCode() method to fix issue # 1
beingtranquil pushed a commit that referenced this issue Mar 17, 2024
@krispatel1001
Revert "Updated ChartView" (#15)
7e89c26 
I am Reverting the changes, I made it by mistake
beingtranquil pushed a commit that referenced this issue Mar 17, 2024
@parikhalay
Merge pull request #15 from vhd1/issue14-remove-this-unused-hasstatus…
3abc224 
…-local-variable

issue14-remove-this-unused-hasstatus-local-variable
beingtranquil pushed a commit that referenced this issue Mar 17, 2024
@SizzlingHot
Merge pull request #15 from Reva-B98/main_dimitri
0fb0d13 
Updated SettingsCustomLayoutEditActivity.java for Issue #7
beingtranquil pushed a commit that referenced this issue Mar 17, 2024
@numanSlm
Merge pull request #15 from numanSlm/branch_parth
9a47ddc 
Update Distance.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Group 09 Group_9_SCM_Project Technical Debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sharanyu @WaseemJan063