Closed (fixed)
Project:
Age Calculator
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
18 Feb 2017 at 11:45 UTC
Updated:
15 Jan 2018 at 12:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dishabhadra commentedReview my patch.
Comment #3
ashishdalviHi @Disha,
Deletion of variables are proper. +1 for RTBC.
But patch have few more changes in age_calculator.helper_functions.inc files. Are these changes are scope of issues?
If no, then it should not be a part of patch. If yes, then we can update the issue summary.
Suggestion on age_calculator.helper_function.inc
We can create a services for helper function. So it will be more usable.
Comment #4
dishabhadra commentedHi @Ashish,
Only variable deletion was the part of this issue.
Maybe in next release, we can consider the suggestion and try to create the Services for helper function.
Comment #5
dishabhadra commentedAs I have done the research and found that in D8 no need to write the hook_uninstall for configuration delete.
It will automatically delete configuration when the module is uninstalled.
So I am removing that code and providing a patch.
Review it.
Comment #7
kunalkursija commentedThanks for the #5 @dishabhadra.
Have Committed to new 8.x-2.x branch, Will be creating the release soon.
Comment #8
kunalkursija commentedThank you for the contribution guys.
I have added a new release that fixes this issue.
Here is 8.x-2.0 release: https://www.drupal.org/project/age_calculator/releases/8.x-2.0.