A debug line seems to have been left in file 'adminimal_admin_menu.js'.

console.log($("#admin-menu-icon > li > a"));
CommentFileSizeAuthor
#6 Screenshot from 2018-01-19 12:18:07.png9.52 KBharshita29
#4 remove_console_log_message-2906227-4.patch4.46 KBsamvel
#2 adminimal_admin_menu-remove_console_log_message-2906227-2.patch599 bytesChris Charlton

Comments

Chris Charlton created an issue. See original summary.

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented
StatusFileSize
new adminimal_admin_menu-remove_console_log_message-2906227-2.patch599 bytes

Patch removes the debug console.log message.

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented
Status: Active » Needs review
samvel’s picture

samvel commented
StatusFileSize
new remove_console_log_message-2906227-4.patch4.46 KB

Hi,

Seems here we should not only remove console.log. There are also code format errors and typo. See attached patch.

harshita29’s picture

harshita29 commented
Assigned: Unassigned » harshita29
harshita29’s picture

harshita29 commented
Assigned: harshita29 » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2018-01-19 12:18:07.png9.52 KB

Hi @Samvel
I have applied your patch it works fine and removed coding standard issue and console.log.

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Reviewed & tested by the community » Fixed

Patch #2 was the correct fix, thanks!

tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Fixed » Closed (fixed)