Skip to content

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