Showing 2,846 of 2,846 total issues
The variable $xp_config is not named in camelCase. Open
Open
public function assignResults(\XoopsTpl $tpl): void
{
$myts = \MyTextSanitizer::getInstance();
$xuEndTimestamp = \xoops_getUserTimestamp($this->pollObj->getVar('end_time'));
$xuEndFormatted = \ucfirst(\date(_MEDIUMDATESTRING, (int)$xuEndTimestamp));- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $subscribe_from is not named in camelCase. Open
Open
public function item(string $subscribe_from)
{
xoops_load('XoopsRequest');
$poll_id = XoopsRequest::getInt('poll_id', 0);
switch ($subscribe_from) {- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The variable $xp_module is not named in camelCase. Open
Open
public function assignResults(\XoopsTpl $tpl): void
{
$myts = \MyTextSanitizer::getInstance();
$xuEndTimestamp = \xoops_getUserTimestamp($this->pollObj->getVar('end_time'));
$xuEndFormatted = \ucfirst(\date(_MEDIUMDATESTRING, (int)$xuEndTimestamp));- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}Source
The method _delete is not named in camelCase. Open
Open
private function _delete($post, $force = false)
{
if ((!$post instanceof Post) || (0 === $post->getVar('post_id'))) {
return false;
}- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method _delete is not named in camelCase. Open
Open
public function _delete($post, $force = false)
{
global $xoopsModule;
if (!is_object($post) || 0 == $post->getVar('post_id')) {- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
TODO found Open
Open
// TODO: cleaning up and merge with post hanldings in viewpost.php- Exclude checks
TODO found Open
Open
//@TODO: add error check here- Exclude checks
TODO found Open
Open
/*@TODO:: Change block templates to use Smarty html_options to support this... then comment- Exclude checks
TODO found Open
Open
//@TODO: test replacement of XoopsTree with XoopsObjectTree- Exclude checks
TODO found Open
Open
TODO if you want to use the render code below,- Exclude checks
TODO found Open
Open
// attachment functions TODO: there should be a file/attachment management class- Exclude checks
TODO found Open
Open
* TODO: combining viewtopic.php- Exclude checks
TODO found Open
Open
* @TODO: Refactor to consider HTML5 & void (self-closing) elements- Exclude checks
TODO found Open
Open
\curl_setopt($curlHandle, \CURLOPT_SSL_VERIFYPEER, true); //TODO: how to avoid an error when 'Peer's Certificate issuer is not recognized'- Exclude checks
HACK found Open
Open
// $votedPolls = []; //TESTING HACK TO BYPASS COOKIES- Exclude checks
TODO found Open
Open
//TODO Parameter '$voterGrps' type is not compatible with declaration- Exclude checks
TODO found Open
Open
TODO: Planned features in the future- Exclude checks
TODO found Open
Open
//@TODO: add error checking here- Exclude checks
TODO found Open
Open
* @TODO need to exit more gracefully on error. Should throw/trigger error and then return false- Exclude checks
TODO found Open
Open
// attachment functions TODO: there should be a file/attachment management class- Exclude checks