View GitHub Repository XoopsCore

Schedule Inspection
Subscribe

Code Duplication    Length = 10-10 lines in 2 locations

htdocs/install/include/createconfigform.php 2 locations

@@ 65-74 (lines=10) @@
62 
                }
63 
                break;
64 

                                

                                                            

                                    65
                                    
                                    
            case 'select':

                                

                                                            

                                    66
                                    
                                    
                $ele = new Xoops\Form\Select($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput());

                                

                                                            

                                    67
                                    
                                    
                $options =& $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));

                                

                                                            

                                    68
                                    
                                    
                $opcount = count($options);

                                

                                                            

                                    69
                                    
                                    
                for ($j = 0; $j < $opcount; ++$j) {

                                

                                                            

                                    70
                                    
                                    
                    $optval = \Xoops\Locale::translate($options[$j]->getVar('confop_value'), 'system');

                                

                                                            

                                    71
                                    
                                    
                    $optkey = \Xoops\Locale::translate($options[$j]->getVar('confop_name'), 'system');

                                

                                                            

                                    72
                                    
                                    
                    $ele->addOption($optval, $optkey);

                                

                                                            

                                    73
                                    
                                    
                }

                                

                                                            

                                    74
                                    
                                    
                break;

                                

                                                            

                                    75
                                    
                                    

                                

                                                            

                                    76
                                    
                                    
            case 'select_multi':

                                

                                                            

                                    77
                                    
                                    
                $ele = new Xoops\Form\Select($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput(), 5, true);

                                

                                                                                

                                
                                
                                    @@ 76-85 (lines=10) @@
                                
                            


                                                            

                                    73
                                    
                                    
                }

                                

                                                            

                                    74
                                    
                                    
                break;

                                

                                                            

                                    75
                                    
                                    

                                

                                                            

                                    76
                                    
                                    
            case 'select_multi':

                                

                                                            

                                    77
                                    
                                    
                $ele = new Xoops\Form\Select($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput(), 5, true);

                                

                                                            

                                    78
                                    
                                    
                $options =& $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));

                                

                                                            

                                    79
                                    
                                    
                $opcount = count($options);

                                

                                                            

                                    80
                                    
                                    
                for ($j = 0; $j < $opcount; ++$j) {

                                

                                                            

                                    81
                                    
                                    
                    $optval = \Xoops\Locale::translate($options[$j]->getVar('confop_value'), 'system');

                                

                                                            

                                    82
                                    
                                    
                    $optkey = \Xoops\Locale::translate($options[$j]->getVar('confop_name'), 'system');

                                

                                                            

                                    83
                                    
                                    
                    $ele->addOption($optval, $optkey);

                                

                                                            

                                    84
                                    
                                    
                }

                                

                                                            

                                    85
                                    
                                    
                break;

                                

                                                            

                                    86
                                    
                                    

                                

                                                            

                                    87
                                    
                                    
            case 'yesno':

                                

                                                            

                                    88
                                    
                                    
                $ele = new Xoops\Form\RadioYesNo($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput(), XoopsLocale::YES, XoopsLocale::NO);