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 empty message in newer Sublime Text 4 versions #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dsego
Copy link

@dsego dsego commented Oct 26, 2023

The new_file.insert method was misusing the edit parameter. Instead we can invoke a command to append text in the newly created window.

Context:

The edit parameter is not for new_file but for self.view.

Build 4158
https://www.sublimetext.com/dev

  • API: Fixed crash when an edit token is passed to the wrong view

API Docs:
https://www.sublimetext.com/docs/api_reference.html#sublime.Edit

Edit objects are passed to TextCommands, and can not be created by the user. Using an invalid Edit object, or an Edit object from a different View, will cause the functions that require them to fail.

My report on Sublime Text bug tracker:
sublimehq/sublime_text#6177

Closes issue #25

The new_file.insert method was misusing the `edit` parameter.
Instead we can invoke a command to append text in the newly created window.

Context:

The edit parameter is not for new_file but for self.view.

Build 4158
https://www.sublimetext.com/dev
-  API: Fixed crash when an edit token is passed to the wrong view

(sublimehq/sublime_text#6177)

API Docs:
https://www.sublimetext.com/docs/api_reference.html#sublime.Edit

> Edit objects are passed to TextCommands, and can not be created by the user. Using an invalid Edit object, or an Edit object from a different View, will cause the functions that require them to fail.
@dsego dsego changed the title Fix empty message in newer Sublime Text versions Fix empty message in newer Sublime Text versions (> 4) Oct 26, 2023
@dsego dsego changed the title Fix empty message in newer Sublime Text versions (> 4) Fix empty message in newer Sublime Text 4 versions Oct 26, 2023
@ronaldmulero
Copy link

This fix works for me using the latest GitCommitMsg in Sublime Text Build 4166. Thanks @dsego !

@mustafa0x
Copy link

helpful, thanks!

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.

3 participants