- Issue created by @Bhupendra_Raykhere
- Issue was unassigned.
- Status changed to Needs review
12 months ago 7:52am 4 July 2023 - 🇮🇳India Bhupendra_Raykhere indore mp
Hello,
I have created a patch to solve all the errors related to phpcs code standard within the .theme file.
Please review and apply it .
Thank You. - Status changed to Needs work
5 months ago 10:25am 6 February 2024 - 🇮🇳India Anjali Mehta
Anjali Mehta → made their first commit to this issue’s fork.
- Assigned to nitin_lama
- 🇮🇳India nitin_lama
Applied patch #2. I can still see there are issues in .theme file.
Remaining issues:
FILE: /home/system/Documents/contribution/portfolio_theme_explore/portfolio_theme_explore.theme ----------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 15 WARNINGS AFFECTING 15 LINES ----------------------------------------------------------------------------------------------- 1260 | WARNING | Unused variable $update. 1269 | WARNING | Unused variable $update. 1288 | WARNING | Unused variable $oldweight. 1342 | WARNING | Unused variable $update. 1351 | WARNING | Unused variable $update. 1370 | WARNING | Unused variable $oldweight. 1393 | WARNING | Unused variable $files. 1394 | WARNING | Unused variable $test. 1399 | WARNING | Unused variable $files. 1400 | WARNING | Unused variable $test. 1409 | WARNING | Messages are user facing text and must run through t() for translation 1417 | WARNING | Messages are user facing text and must run through t() for translation 1426 | WARNING | Messages are user facing text and must run through t() for translation 1433 | WARNING | Unused variable $system_regions. 1436 | WARNING | Unused variable $build. ----------------------------------------------------------------------------------------------- Time: 183ms; Memory: 22MB - Issue was unassigned.