DTD file completion not detected properly

[ejgutierrez74]

I have one file called ( of course in the same directory as xml file) a DTD file called bautistamonicaoriginal.dtd.....In vscode it detects lots of files...except the one i need....

At the same time you wrote the file name, it appear lots of files....

But no one even began with b or ba or bau or baut...and so on....the files which name is helping are far similar to the one i seek

[fbricon]

I can reproduce this weird behavior. When completing on <!DOCTYPE foo SYSTEM "|">, it seems it's listing the contents of the parent directory instead of the current one.

[angelozerr]

@ejgutierrez74 if I remember you need to write ./ before, but I agree it is an annoying issue.

[ejgutierrez74]

@ejgutierrez74 if I remember you need to write ./ before, but I agree it is an annoying issue.

No i dont, im on linux system...in xml copy editor and other like netbeans works ok. I remind that both files are in same directory. I put an Example of changing directory without ./

Also in vscode when you programming in HTML you put:

<link rel="stylesheet" type="text/css" href="CSS/styles.css" />

supposed you have styles.css in CSS folder...no ./ needed

[fbricon]

The issue is caused by https://github.com/eclipse/lemminx/pull/1293/files

The current file completion has some outstanding issues:

• it lists files from the parent directory, when requesting completion for something that doesn't start with a files separator
• it prefixes all results with a /, even when requesting ./something
• applying completion appends the text to the existing value. eg. if you complete "|./foo.dtd" and select the existing "foo.dtd", the result will be "foo.dtd./foo.dtd"

[ejgutierrez74]

The issue is caused by https://github.com/eclipse/lemminx/pull/1293/files

The current file completion has some outstanding issues:

* it lists files from the parent directory, when requesting completion for something that doesn't start with a files separator

* it prefixes all results with a /, even when requesting ./something

* applying completion appends the text to the existing value. eg. if you complete "|./foo.dtd" and select the existing "foo.dtd", the result will be "foo.dtd./foo.dtd"

Its then solved the error ? I see amerged code, dont know if its fixed in new release or nightly or prerelease code....Thanks

[angelozerr]

As soon as the issue is not closed it means that it is not fixed.

We need to create a PR. @fbricon have you the intention to fix it?

[ejgutierrez74]

@fbricon ???

[fbricon]

Sorry I'm busy with other tasks so don't expect me to fix it any time soon.

The problem was introduced in eclipse-lemminx/lemminx#1293. @ejgutierrez74 feel free to open a PR to fix it.

[angelozerr]

Thanks @fbricon for your answer.

I think this file completion must be revisited . I will try to do that to integrate to the next release but I cannot promise you @ejgutierrez74 that I will be able to do that.