Bug fix with username in chat

.gitignore

@@ -3,4 +3,4 @@
 db.sqlite3
 **/migrations/**
 /profiles/static/avatars/*
-!/profiles/static/avatars/default
+!/profiles/static/avatars/default.env

README.md

@@ -22,8 +22,8 @@ pip install -r requirements.txt
 ```
 - Setup database
 ```
-python manage.py runserver makemigrations profiles games
-python manage.py migrate profiles games
+python manage.py runserver makemigrations profiles
+python manage.py migrate profiles
 ```
 - Start the developpement server
 ```

chat/consumers.py

@@ -22,15 +22,15 @@ class ChatConsumer(WebsocketConsumer):
 			self.room_group_name,
 			{
 				'type':'chat_message',
+				'username':self.scope["user"].username,
 				'message':message
 			}
 		)
 
 	def chat_message(self, event):
-		message = event['message']
 
 		self.send(text_data=json.dumps({
 			'type':'chat',
-			'username':self.scope["user"].username,
-			'message':message
+			'username':event['username'],
+			'message':event['message']
 		}))

frontend/static/css/search.css

@@ -0,0 +1,17 @@
+#app img
+{
+	max-height: 100px;
+	max-width: 100px;
+}
+
+#app ul
+{
+	margin: 5px 0 0 0;
+	padding: 0 0 0 0;
+	list-style-type: none;
+}
+
+#app li
+{
+	margin: 10px 10px 0 0;
+}

frontend/static/js/api/client.js

@@ -1,5 +1,4 @@
 import { Account } from "./account.js";
-import { MatchMaking } from "./matchmaking.js";
 import { Profile } from "./profile.js";
 import { Profiles } from "./profiles.js";
 
@@ -20,7 +19,6 @@ class Client
 		this._url = url;
 		this.account = new Account(this);
 		this.profiles = new Profiles(this);
-		this.matchmaking = new MatchMaking(this);
 		this._logged = undefined;
 	}
 

frontend/static/js/api/matchmaking.js

@@ -1,42 +0,0 @@
-import { client, navigateTo } from "../index.js"
-
-class MatchMaking
-{
-	/**
-	 * @param  {client} client
-	 */
-	constructor(client)
-	{
-		/**
-		 * @type {client}
-		 */
-		this.client = client
-	}
-		
-	async start(func)
-	{
-		if (!await this.client.isAuthentificate())
-			return null;
-		
-		console.log(func)
-		this.callback = func
-		console.log(this.callback)
-
-		let url = `wss://${window.location.host}/ws/matchmaking/`;
-		
-		this._chatSocket = new WebSocket(url);
-		
-		this._chatSocket.onmessage = function (event) {
-			const data = JSON.parse(event.data);
-			console.log(func, data)
-			func(data.game_id)
-		};
-	}
-
-	async stop()
-	{
-		this._chatSocket.close()
-	}
-}
-
-export {MatchMaking}

frontend/static/js/api/profile.js

@@ -8,19 +8,19 @@ class Profile
 		this.user_id = user_id
 	}
 
-	async init(user_id)
+	async init(id)
 	{
-		let response = await this.client._get(`/api/profiles/${user_id}`);
+		let response = await this.client._get(`/api/profiles/${id}`);
 		let response_data = await response.json();
 
-		this.user_id = user_id;
+		this.id = id;
 		this.username = response_data.username;
 		this.avatar_url = response_data.avatar_url;
 	}
 
 	async change_avatar(form_data)
 	{
-		let response = await this.client._patch_file(`/api/profiles/${this.user_id}`, form_data);
+		let response = await this.client._patch_file(`/api/profiles/${this.id}`, form_data);
 		let response_data = await response.json()
 
 		return response_data;

frontend/static/js/index.js

@@ -1,6 +1,7 @@
 import LoginView from "./views/accounts/LoginView.js";
 import Dashboard from "./views/Dashboard.js";
 import Settings from "./views/Settings.js";
+import Search from "./views/Search.js";
 import Chat from "./views/Chat.js";
 import HomeView from "./views/HomeView.js";
 import RegisterView from "./views/accounts/RegisterView.js";
@@ -11,7 +12,6 @@ import AbstractRedirectView from "./views/AbstractRedirectView.js";
 import MeView from "./views/MeView.js";
 import ProfilePageView from "./views/profiles/ProfilePageView.js";
 import ProfilesView from "./views/profiles/ProfilesView.js";
-import MatchMakingView from "./views/MatchMakingView.js";
 
 let client = new Client(location.protocol + "//" + location.host)
 
@@ -33,7 +33,7 @@ const navigateTo = async (uri) => {
         history.pushState(null, null, uri);
 };
 
-const router = async (uri) => {
+const router = async (uri = "") => {
     const routes = [
         { path: "/", view: Dashboard },
         { path: "/profiles", view: ProfilesView},
@@ -42,10 +42,10 @@ const router = async (uri) => {
         { path: "/login", view: LoginView },
         { path: "/logout", view: LogoutView },
         { path: "/register", view: RegisterView },
+        { path: "/search", view: Search },
         { path: "/chat", view: Chat },
         { path: "/home", view: HomeView },
         { path: "/me", view: MeView },
-        { path: "/matchmaking", view: MatchMakingView },
     ];
 
     // Test each route for potential match
@@ -86,7 +86,7 @@ const router = async (uri) => {
     return 0;
 };
 
-window.addEventListener("popstate", function() {router(location.pathname)});
+window.addEventListener("popstate", router);
 
 document.addEventListener("DOMContentLoaded", () => {
     document.body.addEventListener("click", e => {

frontend/static/js/views/Chat.js

@@ -4,7 +4,7 @@ export default class extends AbstractAuthentifiedView {
     constructor(params) {
         super(params, "Chat");
 
-		let url = `wss://${window.location.host}/ws/socket-server/`
+		let url = `ws://${window.location.host}/ws/socket-server/`
 
 		this.chatSocket = new WebSocket(url)
 		this.chatSocket.onmessage = function(e){
@@ -44,7 +44,7 @@ export default class extends AbstractAuthentifiedView {
 			<h1>Chat</h1>
 
 			<form id="form">
-				<input type="text" name="message" />
+				<input type="text" name="message" placeholder="message"/>
 			</form>
 
 			<div id="messages">

frontend/static/js/views/HomeView.js

@@ -9,7 +9,6 @@ export default class extends AbstractAuthentificateView {
     async getHtml() {
         return `
 			<h1>HOME</h1>
-            <a href="/matchmaking" class="nav__link" data-link>Play a game</a>
             <a href="/me" class="nav__link" data-link>Me</a>
             <a href="/logout" class="nav__link" data-link>Logout</a>
         `;

frontend/static/js/views/MatchMakingView.js

@@ -1,30 +0,0 @@
-import { client, navigateTo } from "../index.js";
-import AbstractView from "./AbstractView.js";
-
-function game_found(game_id)
-{
-    navigateTo(`/games/${game_id}`)
-}
-
-export default class extends AbstractView {
-    constructor(params) {
-        super(params, "Dashboard");
-    }
-
-    async postInit()
-    {
-        await client.matchmaking.start(game_found)
-        console.log("start matchmaking")
-    }
-
-    async getHtml() {
-        return `
-			<h1>finding<h1>
-        `;
-    }
-
-    async leavePage()
-    {
-		await client.matchmaking.stop();
-    }
-}

frontend/static/js/views/Search.js

@@ -0,0 +1,74 @@
+import AbstractView from "./AbstractView.js";
+import {client} from "../index.js";
+
+export default class extends AbstractView {
+    constructor(params) {
+        super(params, "Search");
+    }
+
+	async postInit() {
+
+		let search = document.getElementById("form");
+		search.addEventListener("input", this.users)
+
+		this.users();
+
+	}
+
+	async users() {
+
+		let search = document.getElementById("form").value;
+		
+		let logged = client.isAuthentificate();
+
+		let users = await client.profiles.all();
+		let list_users = document.getElementById('list_users');
+		list_users.innerHTML = "";
+
+		users.filter(user => user.username.startsWith(search) == true).forEach((user) => {
+			var new_user = document.createElement("li");
+
+			// username
+			let username = document.createElement("a");
+			username.href = `/profiles/${user.user_id}`;
+			username.appendChild(document.createTextNode(user.username));
+			new_user.appendChild(username);
+
+			// space
+			new_user.appendChild(document.createTextNode(" "));
+
+			// chat
+			let chat = document.createElement("a");
+			chat.href = `/chat`
+			chat.appendChild(document.createTextNode("Chat"));
+			new_user.appendChild(chat);
+			
+			// break line
+			new_user.appendChild(document.createElement("br"));
+
+			// avatar
+			var img = document.createElement("img");
+			img.src = user.avatar_url;
+			new_user.appendChild(img);
+
+
+			list_users.appendChild(new_user);
+		});
+		console.log(list_users);
+
+	}
+
+    async getHtml() {
+        return `
+			<link rel="stylesheet" href="/static/css/search.css">
+
+			<input id="form" type="text" name="message" placeholder="userbozo"/>
+
+			<div id="users">
+				<ul id="list_users">
+				</ul>
+			</div>
+
+        `;
+    }
+}

frontend/templates/index.html

@@ -13,7 +13,7 @@
         <a href="/profiles" class="nav__link" data-link>Profiles</a>
         <a href="/login" class="nav__link" data-link>Login</a>
         <a href="/register" class="nav__link" data-link>Register</a>
-        <a href="/chat" class="nav__link" data-link>Chat</a>
+        <a href="/search" class="nav__link" data-link>Search</a>
     </nav>
     <div id="app"></div>
 	<script type="module" src="{% static 'js/index.js' %}"></script>

games/admin.py

@@ -1,3 +0,0 @@
-from django.contrib import admin
-
-# Register your models here.

games/apps.py

@@ -1,6 +0,0 @@
-from django.apps import AppConfig
-
-
-class GamesConfig(AppConfig):
-    default_auto_field = 'django.db.models.BigAutoField'
-    name = 'games'

games/models.py

@@ -1,14 +0,0 @@
-from django.db import models
-
-# Create your models here.
-class GameModel(models.Model):
-	
-	def create(self, users_id: [int]):
-		self.save()
-		for user_id in users_id:
-			GameMembersModel(game_id=self.pk, member_id=user_id)
-		return self.pk
-
-class GameMembersModel(models.Model):
-	game_id = models.IntegerField()
-	member_id = models.IntegerField()

games/tests.py

@@ -1,3 +0,0 @@
-from django.test import TestCase
-
-# Create your tests here.

games/views.py

@@ -1,3 +0,0 @@
-from django.shortcuts import render
-
-# Create your views here.

matchmaking/admin.py

@@ -1,3 +0,0 @@
-from django.contrib import admin
-
-# Register your models here.

matchmaking/apps.py

@@ -1,6 +0,0 @@
-from django.apps import AppConfig
-
-
-class MatchmakingConfig(AppConfig):
-    default_auto_field = 'django.db.models.BigAutoField'
-    name = 'matchmaking'

matchmaking/consumers.py

@@ -1,48 +0,0 @@
-from channels.generic.websocket import WebsocketConsumer
-
-from django.contrib.auth.models import User
-
-from games.models import GameModel
-
-import json
-
-queue_id: [int] = []
-queue_ws: [WebsocketConsumer] = []
-
-class MatchMaking(WebsocketConsumer):
-
-	def __init__(self, *args, **kwargs):
-		super().__init__(*args, **kwargs)
-		self.channel_name = "matchmaking"
-		self.group_name = "matchmaking"
-
-	def connect(self):
-
-		user: User = self.scope["user"]
-		if (user.is_anonymous or not user.is_authenticated):
-			return
-
-		self.channel_layer.group_add(self.group_name, self.channel_name)
-
-		self.accept()
-
-		global queue_id, queue_ws
-		queue_id.append(user.pk)
-		queue_ws.append(self)
-
-		if len(set(queue_id)) == 2:
-			game_id: int = GameModel().create(set(queue_id))
-			event = {"game_id": game_id}
-			for ws in queue_ws:
-				ws.send(text_data=json.dumps({'game_id': game_id}))
-			queue_id.clear()
-			queue_ws.clear()
-
-		
-	def disconnect(self, close_code):
-		user: User = self.scope["user"]
-		global queue_id, queue_ws
-		if (user.pk in queue_id):
-			queue_ws.pop(queue_id.index(user.pk))
-			queue_id.remove(user.pk)
-		self.channel_layer.group_discard(self.group_name, self.channel_name)

matchmaking/models.py

@@ -1,3 +0,0 @@
-from django.db import models
-
-# Create your models here.

matchmaking/routing.py

@@ -1,6 +0,0 @@
-from django.urls import re_path
-from . import consumers
-
-websocket_urlpatterns = [
-	re_path(r'ws/matchmaking/', consumers.MatchMaking.as_asgi())
-]

matchmaking/tests.py

@@ -1,3 +0,0 @@
-from django.test import TestCase
-
-# Create your tests here.

matchmaking/views.py

@@ -1,3 +0,0 @@
-from django.shortcuts import render
-
-# Create your views here.

package-lock.json

@@ -0,0 +1,6 @@
+{
+  "name": "ft_transcendence",
+  "lockfileVersion": 3,
+  "requires": true,
+  "packages": {}
+}

profiles/viewsets.py

@@ -17,7 +17,7 @@ class ProfileViewSet(viewsets.ModelViewSet):
     permission_classes = (permissions.IsAuthenticatedOrReadOnly,)
 
     def retrieve(self, request: HttpRequest, pk=None):
-        instance = ProfileModel.objects.get(pk=pk)
+        instance = self.get_object()
         instance.avatar_url.name = instance.avatar_url.name[instance.avatar_url.name.find("static") - 1:]
         return Response(self.serializer_class(instance).data,
                         status=status.HTTP_200_OK)

trancendence/asgi.py

@@ -10,20 +10,17 @@ https://docs.djangoproject.com/en/4.2/howto/deployment/asgi/
 import os
 from channels.routing import ProtocolTypeRouter, URLRouter
 from channels.auth import AuthMiddlewareStack
-
 import chat.routing
-import matchmaking.routing
 
 from django.core.asgi import get_asgi_application
 
 os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'trancendence.settings')
 
 application = ProtocolTypeRouter({
-	'http':get_asgi_application(),
+    'http':get_asgi_application(),
 	'websocket':AuthMiddlewareStack(
 		URLRouter(
-			chat.routing.websocket_urlpatterns +
-			matchmaking.routing.websocket_urlpatterns
+			chat.routing.websocket_urlpatterns
 		)
 	)
 })

trancendence/settings.py

@@ -43,8 +43,6 @@ INSTALLED_APPS = [
 	'channels',
 	'daphne',
 
-    'matchmaking.apps.MatchmakingConfig',
-    'games.apps.GamesConfig',
     'accounts.apps.AccountsConfig',
     'profiles.apps.ProfilesConfig',
     'frontend.apps.FrontendConfig',