Comments

brahmjeet789 created an issue. See original summary.

brahmjeet789’s picture

Pleas test the code after clean the code.

Status: Needs review » Needs work

The last submitted patch, 2: 2882642-clean-up-code-1.patch, failed testing.

harsha012’s picture

Fixed the short array syntax and $this->t() issue is fixed in this patch

harsha012’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: code-cleanup-2882642-04.patch, failed testing.

harsha012’s picture

harsha012’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 7: code-cleanup-2882642-05.patch, failed testing.

hgunicamp’s picture

I posted a new patch improving the 'code-cleanup-2882642-05.patch' one.

hgunicamp’s picture

Status: Needs work » Needs review
Issue tags: +ciandt-contrib

Status: Needs review » Needs work

The last submitted patch, 10: clean_up_the_codebase-2882642-10.patch, failed testing. View results

hgunicamp’s picture

Sorry. Typo in my previous patch.

Status: Needs review » Needs work

The last submitted patch, 13: clean_up_the_codebase-2882642-13.patch, failed testing. View results