Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

1 - Fix for scopes with new lines #3

Merged
merged 1 commit into from

2 participants

@titoBouzout

Hi!

My first test with this package was to write:

<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title></title>
</head>
<body>
<img src="/
</body>
</html>

I typed manually "<img src="/ and the package failed to show the complementation because view.extract_scope in this situation will return this:

"/
</body>
</html>

This commit fix this by removing everything from the first new line character ( if any )

I'm going to hack this package frequently to improve usage and little things

Thanks!

@BoundInCode
Owner

hmm. I never considered activating it inside an unclosed string. Seems like a 1 in a million issue to me since most people have auto-match quotes on. Oh well, I relent.

@BoundInCode BoundInCode merged commit 13ba8bb into BoundInCode:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2012
  1. @titoBouzout
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  autofilename.py
View
2  autofilename.py
@@ -24,7 +24,7 @@ def on_query_completions(self, view, prefix, locations):
this_dir = os.path.split(view.file_name())[0] + os.path.sep
- cur_path = view.substr(view.extract_scope(sel-1))
+ cur_path = view.substr(view.extract_scope(sel-1)).replace('\r\n', '\n').split('\n')[0]
if cur_path.startswith(("'","\"")):
cur_path = cur_path[1:-1]
Something went wrong with that request. Please try again.