Hi,
Working with stacksight module, Looks great. Reviewed module with coder tool, found some issues ::
Coder found 1 projects, 26 files, 1 critical warnings, 722 normal warnings, 0 warnings were flagged to be ignored.Lot of normal warnings. Issue found as follows :
* Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)
* Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE.
* global variables should start with a single underscore followed by the module and another underscore.
* else statements should begin on a new line
* the use of REQUEST_URI is prone to XSS exploits and does not work on IIS; use request_uri() instead.
Please follow for coding standards
-- https://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/requ...
--
https://www.drupal.org/coding-standards →
--
https://www.drupal.org/taxonomy/term/31450 →
Regards,
Harish B
Closed: outdated
1.0
Code
It involves compliance with, or the content of coding standards. Requires broad community agreement.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
I am closing this issue, which was created for a project version compatible with a Drupal release no longer supported. Feel free to re-open the issue, if it is still relevant for a project version compatible with a supported Drupal release.