Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some YamlGenerator (ModsManager) inconvenience #1131

Merged
merged 4 commits into from
Dec 28, 2024

Conversation

kenjiuno
Copy link
Collaborator

Fix ModsManager crashes on startup when mod.yml is empty file.

Fix Generator of YamlGenerator fails due to null error, if mod.yml is an empty file.

Fix error System.NotImplementedException: Command 17 not implemented yet when trying to decode Japanese evt/sys messages.

Fix problems of ProvideExtractorsService:

  • Remove redundant await Task.Yield();
  • Valid file path to msg.yml is now set:
    - name: msg/jp/sys.bar
      method: binarc
      source:
      - name: sys
        method: kh2msg
        source:
        - name: msg/jp/sys_sys.yml
          language: jp
        type: list
    
  • Japanese evt messages are now set to je like language: je:
    - name: msg/jp/al.bar
      method: binarc
      source:
      - name: sys
        method: kh2msg
        source:
        - name: msg/jp/al_al.yml
          language: je
        type: list
    

… yet` when trying to decode Japanese evt/sys messages.
- Remove redundant `await Task.Yield();`
- Valid file path to msg.yml is now set:
  ```
  - name: msg/jp/sys.bar
    method: binarc
    source:
    - name: sys
      method: kh2msg
      source:
      - name: msg/jp/sys_sys.yml
        language: jp
      type: list
  ```
- Japanese evt messages are now set to `je` like `language: je`:
  ```
  - name: msg/jp/al.bar
    method: binarc
    source:
    - name: sys
      method: kh2msg
      source:
      - name: msg/jp/al_al.yml
        language: je
      type: list
  ```
@shananas shananas merged commit 0f5ba57 into OpenKH:master Dec 28, 2024
3 checks passed
@kenjiuno kenjiuno deleted the fix-mm-msg-extraction branch December 29, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants