From 997bb9524d62c7eacebcd05f5f1ce669658a44b5 Mon Sep 17 00:00:00 2001
From: vtm <vtmilyakov@yandex.ru>
Date: Fri, 12 Jan 2018 00:47:29 +0300
Subject: [PATCH] Improve langs dropdown-selector in gameTab

---
 assets/js/widgets/components/LangSelector.jsx | 21 ++++++-----
 assets/js/widgets/containers/Editor.jsx       | 11 +++---
 .../js/widgets/containers/GameStatusTab.jsx   |  6 ++--
 assets/js/widgets/containers/GameWidget.jsx   |  5 ++-
 assets/js/widgets/middlewares/Game.js         | 35 +++++++++++++------
 lib/mix/tasks/upload_langs.ex                 |  2 ++
 priv/repo/seeds/langs.yml                     |  2 +-
 7 files changed, 50 insertions(+), 32 deletions(-)

diff --git a/assets/js/widgets/components/LangSelector.jsx b/assets/js/widgets/components/LangSelector.jsx
index 36642c892..4503cdb3e 100644
--- a/assets/js/widgets/components/LangSelector.jsx
+++ b/assets/js/widgets/components/LangSelector.jsx
@@ -1,10 +1,13 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import _ from 'lodash';
-import languages from '../config/languages';
+import Gon from 'Gon';
 
-const LangSelector = ({ currentLangKey, onChange }) => {
-  const options = _.filter(_.keys(languages), key => key !== currentLangKey);
+const languages = Gon.getAsset('langs');
+
+const LangSelector = ({ currentLangSlug, onChange }) => {
+
+const [[currentLang, ...other], otherLangs] = _.partition(languages, lang => lang.slug === currentLangSlug);
   return (
     <div className="dropdown">
       <button
@@ -15,19 +18,19 @@ const LangSelector = ({ currentLangKey, onChange }) => {
         aria-haspopup="true"
         aria-expanded="false"
       >
-        {languages[currentLangKey]}
+        {currentLang.name}
       </button>
       <div className="dropdown-menu" aria-labelledby="dropdownLangButton">
-        {_.map(options, langKey => (
+        {_.map(otherLangs, ({ slug, name }) => (
           <button
             className="dropdown-item"
             href="#"
-            key={langKey}
+            key={slug}
             onClick={() => {
-              onChange(langKey);
+              onChange(slug);
             }}
           >
-            {languages[langKey]}
+            {name}
           </button>
         ))}
       </div>
@@ -36,7 +39,7 @@ const LangSelector = ({ currentLangKey, onChange }) => {
 };
 
 LangSelector.propTypes = {
-  currentLangKey: PropTypes.string.isRequired,
+  currentLangSlug: PropTypes.string.isRequired,
   onChange: PropTypes.func.isRequired,
 };
 
diff --git a/assets/js/widgets/containers/Editor.jsx b/assets/js/widgets/containers/Editor.jsx
index f740d9a36..75e34039e 100644
--- a/assets/js/widgets/containers/Editor.jsx
+++ b/assets/js/widgets/containers/Editor.jsx
@@ -1,12 +1,14 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
+import Gon from 'Gon';
 import AceEditor from 'react-ace';
 import 'brace';
 import 'brace/mode/javascript';
 import 'brace/mode/ruby';
 import 'brace/mode/elixir';
 import 'brace/theme/solarized_dark';
-import languages from '../config/languages';
+// import languages from '../config/languages';
+const languages = Gon.getAsset('langs');
 
 const selectionBlockStyle = {
   position: 'absolute',
@@ -21,7 +23,7 @@ class Editor extends Component {
     value: PropTypes.string,
     name: PropTypes.string.isRequired,
     editable: PropTypes.bool,
-    lang: PropTypes.string.isRequired,
+    syntax: PropTypes.string,
     onChange: PropTypes.func,
     allowCopy: PropTypes.bool,
   }
@@ -30,7 +32,7 @@ class Editor extends Component {
     value: '',
     editable: false,
     onChange: null,
-    lang: _.values(languages)[0],
+    syntax: 'javascript',
     allowCopy: true,
   }
 
@@ -39,11 +41,10 @@ class Editor extends Component {
       value,
       name,
       editable,
-      lang,
+      syntax,
       onChange,
       allowCopy,
     } = this.props;
-    const syntax = languages[lang];
 
     return (
       <div style={{ position: 'relative' }}>
diff --git a/assets/js/widgets/containers/GameStatusTab.jsx b/assets/js/widgets/containers/GameStatusTab.jsx
index 488a99268..abc758f5c 100644
--- a/assets/js/widgets/containers/GameStatusTab.jsx
+++ b/assets/js/widgets/containers/GameStatusTab.jsx
@@ -76,10 +76,10 @@ class GameStatusTab extends Component {
                   type="button"
                   disabled
                 >
-                  {languages[leftEditorLang]}
+                  {leftEditorLang.name}
                 </button>
               ) : (
-                <LangSelector currentLangKey={leftEditorLang} onChange={this.props.setLang} />
+                <LangSelector currentLangSlug={leftEditorLang.slug} onChange={this.props.setLang} />
               )}
               {!canCheckResult ? null : (
                 <button
@@ -169,7 +169,7 @@ const mapStateToProps = (state) => {
 
 const mapDispatchToProps = dispatch => ({
   checkResult: () => dispatch(checkGameResult()),
-  setLang: langKey => dispatch(sendEditorLang(langKey)),
+  setLang: langSlug => dispatch(sendEditorLang(langSlug)),
 });
 
 export default connect(mapStateToProps, mapDispatchToProps)(GameStatusTab);
diff --git a/assets/js/widgets/containers/GameWidget.jsx b/assets/js/widgets/containers/GameWidget.jsx
index b7dc80bb5..f2940fa82 100644
--- a/assets/js/widgets/containers/GameWidget.jsx
+++ b/assets/js/widgets/containers/GameWidget.jsx
@@ -38,7 +38,6 @@ class GameWidget extends Component {
     } = this.props;
     // FIXME: currentUser shouldn't return {} for spectator
     const isPlayer = currentUser.type !== userTypes.spectator;
-    console.log(isPlayer, currentUser, currentUser.type , userTypes.spectator)
     const editable = isPlayer;
     const editorState = leftEditor;
     const onChange = isPlayer ?
@@ -48,7 +47,7 @@ class GameWidget extends Component {
     return {
       onChange,
       editable,
-      lang: editorState.currentLang,
+      syntax: _.get(editorState, ['currentLang', 'name'], 'javascript'),
       value: editorState.value,
       name: 'left-editor',
     };
@@ -62,7 +61,7 @@ class GameWidget extends Component {
       onChange: _.noop,
       editable: false,
       allowCopy: false,
-      lang: editorState.currentLang,
+      syntax: _.get(editorState, ['currentLang', 'name'], 'javascript'),
       value: editorState.value,
       name: 'right-editor',
     };
diff --git a/assets/js/widgets/middlewares/Game.js b/assets/js/widgets/middlewares/Game.js
index b16d2ccfe..5d12f6541 100644
--- a/assets/js/widgets/middlewares/Game.js
+++ b/assets/js/widgets/middlewares/Game.js
@@ -1,11 +1,13 @@
-import socket from '../../socket';
+import _ from 'lodash';
 import Gon from 'Gon';
+import socket from '../../socket';
 import { EditorActions, GameActions } from '../redux/Actions';
 import { currentUserIdSelector } from '../selectors/user';
 import { editorsSelector } from '../redux/EditorRedux';
 import userTypes from '../config/userTypes';
 import * as actions from '../actions';
 
+const languages = Gon.getAsset('langs');
 const gameId = Gon.getAsset('game_id');
 const channelName = `game:${gameId}`;
 const channel = socket.channel(channelName);
@@ -24,6 +26,9 @@ const initGameChannel = (dispatch) => {
       task,
     } = response;
 
+    const firstEditorLang = _.find(languages, { slug: first_player_editor_lang });
+    const secondEditorLang = _.find(languages, { slug: second_player_editor_lang });
+
     const users = [{
       id: first_player.id,
       name: first_player.name,
@@ -45,14 +50,14 @@ const initGameChannel = (dispatch) => {
     dispatch(EditorActions.updateEditorData(
       first_player.id,
       first_player_editor_text,
-      first_player_editor_lang,
+      firstEditorLang,
     ));
 
     if (second_player.id) {
       dispatch(EditorActions.updateEditorData(
         second_player.id,
         second_player_editor_text,
-        second_player_editor_lang,
+        secondEditorLang,
       ));
     }
 
@@ -82,12 +87,14 @@ export const sendGiveUp = () => {
   channel.push('give_up');
 };
 
-export const sendEditorLang = lang => (dispatch, getState) => {
+export const sendEditorLang = langSlug => (dispatch, getState) => {
   const state = getState();
   const userId = currentUserIdSelector(state);
-  dispatch(EditorActions.updateEditorLang(userId, lang));
+  const editorLang = _.find(languages, { slug: langSlug });
 
-  channel.push('editor:lang', { lang });
+  dispatch(EditorActions.updateEditorLang(userId, editorLang));
+
+  channel.push('editor:lang', { lang: langSlug });
 };
 
 export const editorReady = () => (dispatch) => {
@@ -96,8 +103,9 @@ export const editorReady = () => (dispatch) => {
     dispatch(EditorActions.updateEditorText(userId, editorText));
   });
 
-  channel.on('editor:lang', ({ user_id: userId, lang }) => {
-    dispatch(EditorActions.updateEditorLang(userId, lang));
+  channel.on('editor:lang', ({ user_id: userId, lang: langSlug }) => {
+    const editorLang = _.find(languages, { slug: langSlug });
+    dispatch(EditorActions.updateEditorLang(userId, editorLang));
   });
 
   channel.on('user:joined', ({
@@ -110,6 +118,11 @@ export const editorReady = () => (dispatch) => {
     second_player_editor_text,
     second_player_editor_lang,
   }) => {
+
+    //TODO: Add strong refactoring
+    const firstEditorLang = _.find(languages, { slug: first_player_editor_lang });
+    const secondEditorLang = _.find(languages, { slug: second_player_editor_lang });
+
     dispatch(GameActions.updateStatus({ status, winner }));
 
     dispatch(actions.updateUsers([{
@@ -127,14 +140,14 @@ export const editorReady = () => (dispatch) => {
     dispatch(EditorActions.updateEditorData(
       first_player.id,
       first_player_editor_text,
-      first_player_editor_lang,
+      firstEditorLang,
     ));
 
     if (second_player.id) {
       dispatch(EditorActions.updateEditorData(
         second_player.id,
         second_player_editor_text,
-        second_player_editor_lang,
+        secondEditorLang,
       ));
     }
   });
@@ -156,7 +169,7 @@ export const checkGameResult = () => (dispatch, getState) => {
 
   const payload = {
     editor_text: currentUserEditor.value,
-    lang: currentUserEditor.currentLang,
+    lang: currentUserEditor.currentLang.slug,
   };
 
   channel.push('check_result', payload)
diff --git a/lib/mix/tasks/upload_langs.ex b/lib/mix/tasks/upload_langs.ex
index fe2423c2b..2bd34009d 100644
--- a/lib/mix/tasks/upload_langs.ex
+++ b/lib/mix/tasks/upload_langs.ex
@@ -10,8 +10,10 @@ defmodule Mix.Tasks.UploadLangs do
 
   def run(_) do
     {:ok, _started} = Application.ensure_all_started(:codebattle)
+
     spec_filepath = Path.join(File.cwd!, "priv/repo/seeds/langs.yml")
     %{langs: langs} = YamlElixir.read_from_file spec_filepath, atoms: true
+
     for lang_data <- langs do
       language = case Repo.get_by(Language, slug: lang_data.slug) do
         nil  -> %Language{slug: lang_data.slug}
diff --git a/priv/repo/seeds/langs.yml b/priv/repo/seeds/langs.yml
index e3a3b1113..5e574456f 100644
--- a/priv/repo/seeds/langs.yml
+++ b/priv/repo/seeds/langs.yml
@@ -6,7 +6,7 @@
     :extension: rb
     :docker_image: "codebattle/ruby:2.5.0"
     :solution_template: "def solution()\n\nend"
-  - :name: js
+  - :name: javascript
     :slug: js
     :version: 9.3.0
     :extension: js