CommentFileSizeAuthor
#2 Removing_unnecessary_2nd_argument_from_config_get_method-3102379-2.patch1.31 KBhardik_patel_12

Comments

Hardik_Patel_12 created an issue. See original summary.

hardik_patel_12’s picture

hardik_patel_12 commented
Status: Active » Needs review
StatusFileSize
new Removing_unnecessary_2nd_argument_from_config_get_method-3102379-2.patch1.31 KB

Kindly follow a new patch.

siddhant.bhosale’s picture

siddhant.bhosale commented
Assigned: Unassigned » siddhant.bhosale
siddhant.bhosale’s picture

siddhant.bhosale commented
Status: Needs review » Reviewed & tested by the community

Hi @Hardik_Patel_12, the code looks good to me and What was previously being done was setting the 2nd argument as default value which is wrong in this case.

siddhant.bhosale’s picture

siddhant.bhosale commented
Assigned: siddhant.bhosale » Unassigned

gaurav.kapoor’s picture

gaurav.kapoor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.