Need to fix notices from CodeSniffer.
https://github.com/podarok/csua/pull/12

CommentFileSizeAuthor
#3 advanced_forum-codesniffer-2364959-2.patch126.87 KBsanchiz
#1 advanced_forum-codesniffer-2364959-1.patch140.61 KBsanchiz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented
Status: Active » Needs review
FileSize
advanced_forum-codesniffer-2364959-1.patch140.61 KB

Initial patch.

Status: Needs review » Needs work

The last submitted patch, 1: advanced_forum-codesniffer-2364959-1.patch, failed testing.

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented
Status: Needs work » Needs review
FileSize
advanced_forum-codesniffer-2364959-2.patch126.87 KB

Updated patch according to latest dev version.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs review » Fixed

#3 commited
Thank You tons!!!!

  • podarok committed cfba428 on 7.x-2.x authored by sanchiz 
    Issue #2364959 by sanchiz: Fix notices from CodeSniffer.

Status: Fixed » Closed (fixed)

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