Öne çıkan içerik

Server: Tidy up logger initialisation Previously this would be manually initialised within App::OnStart(), after config is loaded so that it can fetch the associated logger config. This creates a problem with ftxui/renderer setup; it really should be rendering throughout startup, so we don't have to be awkwardly swapping from 'raw' console back to the ftxui rendered container. To rectify this, we abstract config loading to handle it under AppThread. It loads the config file (which no...

Server: Minor cleanup (#654) * Server: Minor cleanup Move loggers to their own files, remove remaining ReportTableLoadError() refs. * AIServerApp: Rename member vars for consistency * AI: Strip Define.h Github Commit Görüntüle... See Commints... ..

Merge pull request #653 from Open-KO/server/strip-msc-ver Server: Strip _MSC_VER checks Github Commit Görüntüle... See Commints... ..

Server: Strip _MSC_VER checks As-is, they're not portable. No real reason to not just use #pragma once directly; all modern compilers handle this just fine. Github Commit Görüntüle... See Commints... ..

Merge pull request #652 from Open-KO/server/rename-instance-to-app Rename server 'Instances' to 'Apps' to match the 'App'Thread Github Commit Görüntüle... See Commints... ..

Rename server 'Instances' to 'Apps' to match the 'App'Thread Github Commit Görüntüle... See Commints... ..

Merge pull request #651 from Open-KO/server/load-zone-info-first AI: Load ZONE_INFO table explicitly before maps or spawn data Github Commit Görüntüle... See Commints... ..

AI: Load ZONE_INFO table explicitly before maps or spawn data GetServerNumber() should use this for its zone ID -> server ID mapping, so that it's available before maps have loaded. Maps load object event NPCs before they insert to their own container, so the old method simply doesn't work at all for object event NPCs. This fixes the numerous "AiServerInstance::GetServerNumber: zoneId={} not found" errors during server startup. Github Commit Görüntüle... See Commints... ..

Merge pull request #650 from Open-KO/server/fix-signed-overflow-warning Server: Fix signed overflow check warning in myrand_generic() Github Commit Görüntüle... See Commints... ..

Server: Fix signed overflow check warning in myrand_generic() This ensures the result can never overflow and wrap around; if it does, the result ends up becoming max instead. https://github.com/Open-KO/KnightOnline/security/code-scanning/43 Github Commit Görüntüle... See Commints... ..

Merge pull request #649 from Open-KO/use-console-sink-by-default Server: Use the console sink by default Github Commit Görüntüle... See Commints... ..

Add docker as an option for running our latest database (#647) Adds docker as an option for running our latest database * Sets up an mssql server 2022 service `sqlserver` * Has a `kodb-util` service that can be called to perform database operations such as clean imports. It clones the kodb-util tool, builds it, then uses it to import the data from OpenKO-db into the mssql instance. Github Commit Görüntüle... See Commints... ..

Server: Use the console sink by default As OnStart() is what sets up the loggers, and rendering occurs afterwards, we have the ftxui sink just use the console logger to log on-demand as it's loading up until it's finished loading and it sets up the ftxui sink for rendering. We rename _useStdout to _useConsoleSink to be more explicit, and add _storeLogBuffer as everything we're writing to console now should be swapped over to & rendered by ftxui. Github Commit Görüntüle... See Commints... ..

Merge pull request #648 from Open-KO/server/remove-dx9-refs Remove DX9 refs & build dependencies from server projects Github Commit Görüntüle... See Commints... ..

Remove DX9 refs & build dependencies from server projects Github Commit Görüntüle... See Commints... ..

Merge pull request #643 from Open-KO/remove-mfc/Ebenezer Remove MFC/UI: Ebenezer Github Commit Görüntüle... See Commints... ..

Implement FTXUI 6.1.9 for command & hotkey handling Github Commit Görüntüle... See Commints... ..

Remove StringConversion helpers, move DebugUtils to N3Base StringConversion helpers were used by CIni (which no longer bothers to use wide-char variants, so translation is unnecessary), and various path fetches in the server. This was looked at here because of the path translations; they were inconsistent, not always re-encoding appropriately. Now they're all consistent and this has been removed. StringConversion also depended on Windows (MultiByteToWideChar/WideCharToMultiByte calls), so...

Remove MFC/UI: Ebenezer With the exception of server command handling (which is temporarily disabled), everything is swapped to behave in a console. Github Commit Görüntüle... See Commints... ..

Merge pull request #642 from Open-KO/remove-mfc/Aujard Remove MFC/UI: Aujard Github Commit Görüntüle... See Commints... ..