Mahdi / Fixed MT5 trade server selection bug (#6771)

* Fixed trade server selector bug

* Refactored checking sections

Author: mahdiaryayi-fs

src/javascript/app/pages/user/metatrader/metatrader.ui.js

@@ -125,7 +125,7 @@ const MetaTraderUI = (() => {
 
                     $ddl_trade_server
                         .append(
-                            $('<div />', { id: trading_server.id, class: 'gr-padding-10 gr-parent' })
+                            $('<div />', { id: trading_server.id, class: 'gr-padding-10 gr-parent trade-server-item' })
                                 .append($('<input />', input_attributes))
                                 .append($('<label />', { htmlFor: trading_server.id })
                                     .append($('<span />', { text: label_text }))
@@ -821,16 +821,13 @@ const MetaTraderUI = (() => {
         });
 
         $form.find('#ddl_trade_server').off('click').on('click', (e) => {
-            $form.find('#ddl_trade_server').find('input').not(':input[disabled]').removeAttr('checked');
+            const $target_input = $(e.target).parents('.trade-server-item').children('input');
 
-            if (e.target.nodeName === 'SPAN') {
-                $(e.target.parentElement).parent().find('input').not(':input[disabled]').attr('checked', 'checked');
-            }
-            if (e.target.nodeName === 'LABEL') {
-                $(e.target.parentElement).find('input').not(':input[disabled]').attr('checked', 'checked');
-            }
-            if (e.target.nodeName === 'INPUT') {
-                $(e.target).not(':input[disabled]').attr('checked', 'checked');
+            if ($target_input.attr('disabled')) return;
+
+            if ($target_input.length) {
+                $form.find('#ddl_trade_server').find('input').not(':input[disabled]').removeAttr('checked');
+                $target_input.not(':input[disabled]').attr('checked', 'checked');
             }
 
             const new_user_submit_button = $form.find('#new_user_btn_submit_new_account');