YetiForceCompany/YetiForceCRM

Vtiger_Widget_Model has 32 functions (exceeds 20 allowed). Consider refactoring.

Open

class Vtiger_Widget_Model extends \App\Base
{
    /** @var array Default labels */
    const DEFAULT_LABELS = [
        'Activities' => 'LBL_ACTIVITIES',

Identical blocks of code found in 3 locations. Consider refactoring.

Open

    setCalendarModuleOptions() {
        let options = super.setCalendarModuleOptions();
        options.selectable = true;
        options.eventClick = function (info) {
            info.jsEvent.preventDefault();

• Read up
  Read up

Identical blocks of code found in 3 locations. Consider refactoring.

Open

    setCalendarModuleOptions() {
        let options = super.setCalendarModuleOptions();
        options.selectable = true;
        options.eventClick = function (info) {
            info.jsEvent.preventDefault();

• Read up
  Read up

Identical blocks of code found in 3 locations. Consider refactoring.

Open

    setCalendarModuleOptions() {
        let options = super.setCalendarModuleOptions();
        options.selectable = true;
        options.eventClick = function (info) {
            info.jsEvent.preventDefault();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (parentModule == 'Settings') {
                    moduleClassName = parentModule + '_' + module + '_List_Js';
                    if (typeof window[moduleClassName] === 'undefined') {
                        moduleClassName = module + '_List_Js';
                    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if (parentModule === 'Settings') {
                moduleClassName = parentModule + '_' + moduleName + '_Edit_Js';
                if (typeof window[moduleClassName] === 'undefined') {
                    moduleClassName = moduleName + '_Edit_Js';
                }

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                foreach ($permissionsRelatedField as $row) {
                    if (!$relatedPermission) {
                        switch ($row) {
                            case 0:
                                $relatedPermission = $recordMetaData['smownerid'] == $currentUserId || \in_array($recordMetaData['smownerid'], $currentUserGroups);

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                foreach ($permissionsRelatedField as $row) {
                    if (!$relatedPermission) {
                        switch ($row) {
                            case 0:
                                $relatedPermission = $recordMetaData['smownerid'] == $currentUserId || \in_array($recordMetaData['smownerid'], $currentUserGroups);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function getListQuery(): App\Db\Query
    {
        $queryGenerator = new App\QueryGenerator('Faq');
        $queryGenerator->setFields(['id', 'assigned_user_id', 'subject', 'introduction', 'modifiedtime', 'category']);
        $queryGenerator->addNativeCondition(['vtiger_faq.status' => 'Published']);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function getListViewURL($user = false)
    {
        $url = 'index.php?module=' . $this->getTargetModule() . '&view=List&viewname=' . $this->widgetModel->get('filterid');
        if (!$user) {
            $user = App\User::getCurrentUserId();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function getListQuery(): App\Db\Query
    {
        $queryGenerator = new App\QueryGenerator('KnowledgeBase');
        $queryGenerator->setFields(['id', 'assigned_user_id', 'subject', 'introduction', 'modifiedtime', 'category']);
        $queryGenerator->addNativeCondition(['knowledgebase_status' => 'PLL_ACCEPTED']);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function getTotalCountURL($user = false)
    {
        $url = 'index.php?module=' . $this->getTargetModule() . '&action=Pagination&mode=getTotalCount&viewname=' . $this->widgetModel->get('filterid');
        if (!$user) {
            $user = App\User::getCurrentUserId();

• Read up
  Read up

Function loadFieldSpecificUi has 99 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        loadFieldSpecificUi: function (fieldSelect) {
            const selectedOption = fieldSelect.find('option:selected');
            const row = fieldSelect.closest('div.js-conditions-row');
            const fieldUiHolder = row.find('.fieldUiHolder');
            const conditionSelectElement = row.find('select[name="comparator"]');

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        registerVariablePanelEvent: function (form) {
            if (typeof form === 'undefined') {
                form = this.getForm();
            }
            form

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        registerVariablePanelEvent: function (form) {
            if (typeof form === 'undefined') {
                form = this.getForm();
            }
            form

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    if (excludedIds != '') {
                        jQuery('#listViewEntriesMainCheckBox').prop('checked', false);
                        jQuery('.listViewEntriesCheckBox').each(function (index, element) {
                            if (jQuery.inArray(jQuery(element).val(), excludedIds) != -1) {
                                jQuery(element).prop('checked', false).closest('tr').removeClass('highlightBackgroundColor');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    if (excludedIds != '') {
                        $('#listViewEntriesMainCheckBox').prop('checked', false);
                        $('.listViewEntriesCheckBox').each(function (index, element) {
                            if ($.inArray($(element).val(), excludedIds) != -1) {
                                $(element).prop('checked', false).closest('tr').removeClass('highlightBackgroundColor');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        this.files = actualElements.map((element) => {
            for (let i = 0, len = this.files.length; i < len; i++) {
                const elementHash = $(element).data('hash');
                if (this.files[i].hash === elementHash) {
                    return this.files[i];

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            this.files = actualElements.map((element) => {
                for (let i = 0, len = this.files.length; i < len; i++) {
                    const elementHash = $(element).data('key');
                    if (this.files[i].key === elementHash) {
                        return this.files[i];

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function addOrRemoveMembers(App\Request $request)
    {
        $module = $request->getInteger('srcModule');
        $members = $request->getArray('members', 'Text');
        $state = $request->getBoolean('isToAdd') ? 1 : 0;

• Read up
  Read up