mambax7 / alumni-26x

index.php

<?php
/*
 You may not change or alter any portion of this comment or credits
 of supporting developers from this source code or any supporting source code
 which is considered copyrighted (c) material of the original comment or credit authors.

 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*/

/**
 * Alumni module for Xoops
 *
 * @copyright       XOOPS Project https://xoops.org/
 * @license         GPL 2.0 or later
 * @package         alumni
 * @since           2.6.x
 * @author          John Mordo (jlm69)
 */

use Xoops\Core\Request;

include __DIR__ . '/header.php';

$moduleDirName = basename(__DIR__);
$main_lang     = '_MA_' . strtoupper($moduleDirName);

$main_lang does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$myts          = MyTextSanitizer::getInstance();
$xoops         = Xoops::getInstance();
$module_id     = $xoops->module->getVar('mid');

$module_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$groups = '3';
if (is_object($xoops->user)) {
    $groups = $xoops->user->getGroups();
}
//$gperm_handler = $xoops->getHandler('groupperm');

64% of this comment could be valid code. Did you maybe forget this after debugging?

$perm_itemid = 0;

$perm_itemid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (isset($_POST['item_id'])) {
    $perm_itemid = (int)$_POST['item_id'];

$perm_itemid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}
//If no access
if (!$gpermHandler->checkRight('' . $moduleDirName . '_view', $perm_itemid, $groups, $module_id)) {

$perm_itemid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

This code seems to be duplicated across your project.

    $xoops->redirect(XOOPS_URL . '/index.php', 3, XoopsLocale::E_NO_ACCESS_PERMISSION);
    exit();
}
$prem_perm = '1';

$prem_perm does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (!$gpermHandler->checkRight('' . $moduleDirName . '_premium', $perm_itemid, $groups, $module_id)) {

$perm_itemid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $prem_perm = '0';

$prem_perm does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

$alumni = Alumni::getInstance();

$gpermHandler = $xoops->getHandlerGroupPermission();

//        $alumni_user = $gperm_handler->getItemIds('alumni_view', $xoops->user->getGroups(), $module_id);

62% of this comment could be valid code. Did you maybe forget this after debugging?

//        $alumni_premium = $gperm_handler->getItemIds('alumni_submit', $xoops->user->getGroups(), $module_id);

//	if (!$alumni_user) {

52% of this comment could be valid code. Did you maybe forget this after debugging?

//	$xoops->redirect(XOOPS_URL . "/user.php", 3, _NOPERM);
//	}
$totalCategories = $alumni->getCategoryHandler()->getCategoriesCount(0);

// if there ain't no category to display, let's get out of here
if (0 == $totalCategories) {
    $xoops->redirect(\XoopsBaseConfig::get('url'), 12, _NOPERM);
}

$xoops->header('module:alumni/alumni_index.tpl');
Xoops::getInstance()->header();
$xoops->tpl()->assign('xmid', $xoopsModule->getVar('mid'));
$xoops->tpl()->assign('add_from', AlumniLocale::ALUMNI_LISTINGS . ' ' . $xoopsConfig['sitename']);
$xoops->tpl()->assign('add_from_sitename', $xoopsConfig['sitename']);
$xoops->tpl()->assign('add_from_title', AlumniLocale::ALUMNI_LISTINGS);
$xoops->tpl()->assign('class_of', AlumniLocale::CLASSOF);
$xoops->tpl()->assign('front_intro', AlumniLocale::FINTRO);

if ('1' == $xoops->getModuleConfig('' . $moduleDirName . '_offer_search')) {
    $xoops->tpl()->assign('offer_search', true);
    $xoops->tpl()->assign('search_listings', AlumniLocale::SEARCH_LISTINGS);
    $xoops->tpl()->assign('match', AlumniLocale::MATCH);
    $xoops->tpl()->assign('all_words', AlumniLocale::ALL_WORDS);
    $xoops->tpl()->assign('any_words', AlumniLocale::ANY_WORDS);
    $xoops->tpl()->assign('exact_match', AlumniLocale::EXACT_MATCH);
    $xoops->tpl()->assign('byyear', AlumniLocale::BYYEAR);
    $xoops->tpl()->assign('bycategory', AlumniLocale::BYCATEGORY);
    $xoops->tpl()->assign('keywords', XoopsLocale::KEYWORDS);
    $xoops->tpl()->assign('search', XoopsLocale::SEARCH);

    $categoriesHandler = $xoops->getModuleHandler('category', 'alumni');

    $alumni       = Alumni::getInstance();
    $helper       = Xoops::getModuleHelper('alumni');
    $module_id    = $helper->getModule()->getVar('mid');

$module_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $groups       = $xoops->isUser() ? $xoops->user->getGroups() : '3';
    $alumni_ids   = $xoops->getHandlerGroupPermission()->getItemIds('alumni_view', $groups, $module_id);

$alumni_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $cat_criteria = new CriteriaCompo();

$cat_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $cat_criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$cat_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $cat_criteria->setOrder('' . $xoops->getModuleConfig('' . $moduleDirName . '_csortorder') . '');

$cat_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $numcat       = $categoriesHandler->getCount($cat_criteria);

$cat_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $category_arr = $categoriesHandler->getAll($cat_criteria);

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    foreach (array_keys($category_arr) as $i) {

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $cid      = $category_arr[$i]->getVar('cid');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $pid      = $category_arr[$i]->getVar('pid');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $title    = $category_arr[$i]->getVar('title', 'e');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $img      = $category_arr[$i]->getVar('img');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $order    = $category_arr[$i]->getVar('ordre');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $affprice = $category_arr[$i]->getVar('affprice');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $title    = $myts->htmlSpecialChars($title);
        $xoops->tpl()->assign('title', $title);
    }

    include_once XOOPS_ROOT_PATH . "/modules/{$moduleDirName}/class/alumni_tree.php";
    $cattree = new AlumniObjectTree($category_arr, 'cid', 'pid');

$category_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $categories      = $alumni->getCategoryHandler()->getCategoriesForSearch();
    $by_cat          = Request::getInt('by_cat');

$by_cat does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $select_category = '<select name="by_cat">';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $select_category .= '<option value="all"';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if (empty($by_cat) || 0 == count($by_cat)) {

$by_cat does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $select_category .= 'selected="selected"';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    $select_category .= '>' . XoopsLocale::ALL . '</option>';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    foreach ($categories as $cid => $title) {
        $select_category .= '<option value="' . $cid . '"';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        if ($cid = $by_cat) {

$by_cat does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $select_category .= 'selected="selected"';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
        $select_category .= '>' . $title . '</option>';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    $select_category .= '</select>';

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $xoops->tpl()->assign('category_select', $select_category);

$select_category does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

$index_banner = $xoops->getBanner();

$index_banner does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('index_banner', $index_banner);

$index_banner does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$index_code_place = $xoops->getModuleConfig('' . $moduleDirName . '_code_place');

$index_code_place does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$use_extra_code   = $xoops->getModuleConfig('' . $moduleDirName . '_use_code');

$use_extra_code does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$use_banner       = $xoops->getModuleConfig('' . $moduleDirName . '_use_banner');

$use_banner does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$index_extra_code = $xoops->getModuleConfig('' . $moduleDirName . '_index_code');

$index_extra_code does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('use_extra_code', $use_extra_code);

$use_extra_code does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('use_banner', $use_banner);

$use_banner does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('index_extra_code', '<html>' . $index_extra_code . '</html>');

$index_extra_code does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('index_code_place', $index_code_place);

$index_code_place does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoops->tpl()->assign('moduleDirName', $moduleDirName);

$cats  = $cattree->alumni_getFirstChild(0, $alumni_ids);

$alumni_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$count = 0;

foreach (array_keys($cats) as $i) {
    if (in_array($cats[$i]->getVar('cid'), $alumni_ids)) {

$alumni_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $cat_img = $cats[$i]->getVar('img');

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        if ('http://' !== $cat_img) {

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $cat_img = XOOPS_URL . "/modules/{$moduleDirName}/assets/images/cat/$cat_img";

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        } else {
            $cat_img = '';

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }

        $listingHandler = $xoops->getModuleHandler('listing', 'alumni');
        $count_criteria = new CriteriaCompo();

$count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $count_criteria->add(new Criteria('cid', $cats[$i]->getVar('cid'), '='));

$count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $count_criteria->add(new Criteria('valid', 1, '='));

$count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $count_criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $listings = $listingHandler->getCount($count_criteria);

$count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $publishdate = isset($listings['date'][$cats[$i]->getVar('cid')]) ? $listings['date'][$cats[$i]->getVar('cid')] : 0;
        $all_subcats = $cattree->alumni_getAllChild($cats[$i]->getVar('cid'));

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        if (count($all_subcats) > 0) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            foreach (array_keys($all_subcats) as $k) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                if (in_array($all_subcats[$k]->getVar('cid'), $alumni_ids)) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                    $publishdate = (isset($listings['date'][$all_subcats[$k]->getVar('cid')]) and $listings['date'][$all_subcats[$k]->getVar('cid')] > $publishdate) ? $listings['date'][$all_subcats[$k]->getVar('cid')] : $publishdate;

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

Using logical operators such as and instead of && is generally not recommended.

                }
            }
        }
    }
    $subcategories = [];

    $count++;

    $listingHandler   = $xoops->getModuleHandler('listing', 'alumni');
    $listing_criteria = new CriteriaCompo();

$listing_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $listing_criteria->add(new Criteria('cid', $cats[$i]->getVar('cid'), '='));

$listing_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $listing_criteria->add(new Criteria('valid', 1, '='));

$listing_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $listing_criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$listing_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $alumni_count = $listingHandler->getCount($listing_criteria);

$alumni_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if (count($all_subcats) > 0) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        foreach (array_keys($all_subcats) as $k) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            if (in_array($all_subcats[$k]->getVar('cid'), $alumni_ids)) {

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $listingHandler     = $xoops->getModuleHandler('listing', 'alumni');
                $sub_count_criteria = new CriteriaCompo();

$sub_count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $sub_count_criteria->add(new Criteria('cid', $all_subcats[$k]->getVar('cid'), '='));

$sub_count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $sub_count_criteria->add(new Criteria('valid', 1, '='));

$sub_count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $sub_count_criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$sub_count_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $alumni_subcount = $listingHandler->getCount($sub_count_criteria);

$alumni_subcount does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                if (1 == $xoops->getModuleConfig('alumni_showsubcat') and $all_subcats[$k]->getVar('pid') == $cats[$i]->getVar('cid')) { // if we are collecting subcat info for displaying, and this subcat is a first level child...

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

Using logical operators such as and instead of && is generally not recommended.

                    $subcategories[] = ['id' => $all_subcats[$k]->getVar('cid'), 'title' => $all_subcats[$k]->getVar('title'), 'count' => $alumni_subcount];

$all_subcats does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                }
            }
        }
    }

    if (1 != $xoops->getModuleConfig('alumni_showsubcat')) {
        unset($subcategories);

        $xoops->tpl()->append('categories', [
            'image'     => $cat_img,

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            'id'        => (int)$cats[$i]->getVar('cid'),
            'title'     => $cats[$i]->getVar('title'),
            'totalcats' => $alumni_count,

$alumni_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            'count'     => $count
        ]);
    } else {
        $xoops->tpl()->append('categories', [
            'image'         => $cat_img,

$cat_img does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            'id'            => (int)$cats[$i]->getVar('cid'),
            'title'         => $cats[$i]->getVar('title'),
            'subcategories' => $subcategories,
            'totalcats'     => $alumni_count,

$alumni_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            'count'         => $count
        ]);
    }
}
$xoops->tpl()->assign('total_confirm', '');

$listingHandler = $xoops->getModuleHandler('listing', 'alumni');

$xoops->tpl()->assign('moderated', false);
if ('1' == $xoops->getModuleConfig('' . $moduleDirName . '_moderated')) {
    $xoops->tpl()->assign('moderated', true);
    $moderate_criteria = new CriteriaCompo();

$moderate_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $moderate_criteria->add(new Criteria('valid', 0, '='));

$moderate_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $moderate_criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$moderate_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $moderate_rows = $listingHandler->getCount($moderate_criteria);

$moderate_rows does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $moderate_arr  = $listingHandler->getAll($moderate_criteria);

$moderate_arr does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if ($xoops->isUser()) {
        if ($xoops->user->isAdmin()) {
            $xoops->tpl()->assign('user_admin', true);

            $xoops->tpl()->assign('admin_block', AlumniLocale::ADMIN_PANEL);
            if (0 == $moderate_rows) {

$moderate_rows does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

This code seems to be duplicated across your project.

                $xoops->tpl()->assign('confirm_alumni', AlumniLocale::NO_LISTING_TO_APPROVE);
            } else {
                $xoops->tpl()->assign('confirm_alumni', AlumniLocale::THERE_ARE . " $moderate_rows  " . AlumniLocale::WAITING . '<br><a href="admin/alumni.php?op=list_moderated">' . constant($main_lang . '_SEEIT') . '</a>');

$moderate_rows does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            }
            $xoops->tpl()->assign('total_confirm', AlumniLocale::THIS_AND . " $moderate_rows " . AlumniLocale::WAITING);

$moderate_rows does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
    }
}

$criteria = new CriteriaCompo();
$criteria->add(new Criteria('valid', 1, '='));
$criteria->add(new Criteria('cid', '(' . implode(', ', $alumni_ids) . ')', 'IN'));

$alumni_ids does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$criteria->setLimit($xoops->getModuleConfig('' . $moduleDirName . '_per_page'));
$numrows = $listingHandler->getCount($criteria);

$xoops->tpl()->assign('total_listings', AlumniLocale::THERE_ARE . ' ' . $numrows . ' ' . AlumniLocale::ALUMNI_LISTINGS . ' ' . AlumniLocale::IN . ' ' . $numcat . ' ' . AlumniLocale::CATEGORIES);
$xoops->tpl()->assign('last_head', AlumniLocale::THE . ' ' . $xoops->getModuleConfig('' . $moduleDirName . '_newalumni') . ' ' . AlumniLocale::LASTADD);
$xoops->tpl()->assign('last_head_name', AlumniLocale::NAME_2);
$xoops->tpl()->assign('last_head_school', AlumniLocale::SCHOOL_2);
$xoops->tpl()->assign('last_head_studies', AlumniLocale::STUDIES_2);
$xoops->tpl()->assign('last_head_year', AlumniLocale::YEAR_2);
$xoops->tpl()->assign('last_head_date', XoopsLocale::DATE);
$xoops->tpl()->assign('last_head_local', AlumniLocale::TOWN_2);
$xoops->tpl()->assign('last_head_views', AlumniLocale::HITS);
$xoops->tpl()->assign('last_head_photo', AlumniLocale::PHOTO);

$listingArray = $listingHandler->getAll($criteria);

foreach (array_keys($listingArray) as $i) {
    $lid        = $listingArray[$i]->getVar('lid');
    $cid        = $listingArray[$i]->getVar('cid');
    $name       = $listingArray[$i]->getVar('name');
    $mname      = $listingArray[$i]->getVar('mname');
    $lname      = $listingArray[$i]->getVar('lname');
    $school     = $listingArray[$i]->getVar('school');
    $year       = $listingArray[$i]->getVar('year');
    $studies    = $listingArray[$i]->getVar('studies');
    $activities = $listingArray[$i]->getVar('activities');
    $extrainfo  = $listingArray[$i]->getVar('extrainfo');
    $occ        = $listingArray[$i]->getVar('occ');
    $date       = $listingArray[$i]->getVar('date');
    $email      = $listingArray[$i]->getVar('email');
    $submitter  = $listingArray[$i]->getVar('submitter');
    $usid       = $listingArray[$i]->getVar('usid');
    $town       = $listingArray[$i]->getVar('town');
    $valid      = $listingArray[$i]->getVar('valid');
    $photo      = $listingArray[$i]->getVar('photo');
    $photo2     = $listingArray[$i]->getVar('photo2');
    $view       = $listingArray[$i]->getVar('view');

    $a_item        = [];

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['new'] = '';

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $newcount  = $xoops->getModuleConfig('' . $moduleDirName . '_countday');
    $startdate = (time() - (86400 * $newcount));
    if ($startdate < $date) {

This code seems to be duplicated across your project.

        $newitem       = '<img src="' . XOOPS_URL . "/modules/{$moduleDirName}/assets/images/newred.gif\">";
        $a_item['new'] = $newitem;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }

    $useroffset = '';
    if ($xoops->user) {

This code seems to be duplicated across your project.

        $timezone = $xoops->user->timezone();
        if (isset($timezone)) {
            $useroffset = $xoops->user->timezone();
        } else {
            $useroffset = $xoopsConfig['default_TZ'];
        }
    }
    $date = ($useroffset * 3600) + $date;

    $date = XoopsLocale::formatTimestamp($date, 's');

    if ($xoops->user) {

This code seems to be duplicated across your project.

        if ($xoops->user->isAdmin()) {
            $a_item['admin'] = "<a href='admin/alumni.php?op=edit_listing&amp;lid=$lid&amp;cid=$cid'><img src='images/modif.gif' border=0 alt=\"" . AlumniLocale::MODADMIN . '"></a>';

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
    }

    $a_item['name']    = "<a href='listing.php?lid=$lid'><b>$name&nbsp;$mname&nbsp;$lname</b></a>";

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['school']  = $school;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['year']    = $year;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['studies'] = $studies;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['date']    = $date;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $a_item['local']   = '';

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if ($town) {
        $a_item['local'] .= $town;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }

    if ($photo) {
        $a_item['photo'] = "<a href=\"javascript:CLA('display-image.php?lid=$lid')\"><img src=\"" . XOOPS_URL . "/modules/{$moduleDirName}/assets/images/photo.gif\" border=\"0\" width=\"15\" height=\"11\" alt='" . AlumniLocale::PHOTO_AVAILABLE . "'></a>";

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    } else {
        $a_item['photo'] = '';

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }

    $a_item['views'] = $view;

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $xoops->tpl()->append('items', $a_item);

$a_item does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

Xoops::getInstance()->footer();