I went through to code for this module. Found in .install file has two tables unnecessarily used. It can be optimised into one

CommentFileSizeAuthor
#7 2975356-6.patch2.7 KBmahtab_alam
#5 2975356-5.patch2.85 KBmahtab_alam
#3 2975356-2.patch2.84 KBmahtab_alam

Comments

chishah92 created an issue. See original summary.

mahtab_alam’s picture

mahtab_alam commented
Assigned: Unassigned » mahtab_alam

I will provide the patch for this

mahtab_alam’s picture

mahtab_alam commented
Status: Active » Needs review
StatusFileSize
new 2975356-2.patch2.84 KB

Patch for this

nishantkumar155’s picture

nishantkumar155 commented
Status: Needs review » Needs work

Patch #3 need work for white space .

Checking patch user_statistics.install...
Checking patch user_statistics.module...
Applied patch user_statistics.install cleanly.
Applied patch user_statistics.module cleanly.
warning: 1 line adds whitespace errors.

mahtab_alam’s picture

mahtab_alam commented
Status: Needs work » Needs review
StatusFileSize
new 2975356-5.patch2.85 KB
nishantkumar155’s picture

nishantkumar155 commented
Status: Needs review » Needs work

Patch #5 have white space .

2975356-5.patch:46: trailing whitespace.
);
Checking patch user_statistics.install...
Checking patch user_statistics.module...
Applied patch user_statistics.install cleanly.
Applied patch user_statistics.module cleanly.
warning: 1 line adds whitespace errors.

mahtab_alam’s picture

mahtab_alam commented
Status: Needs work » Needs review
StatusFileSize
new 2975356-6.patch2.7 KB
nishantkumar155’s picture

nishantkumar155 commented
Status: Needs review » Reviewed & tested by the community

Patch #7 applied successfully .

nishantkumar155’s picture

nishantkumar155 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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