-
Notifications
You must be signed in to change notification settings - Fork 3
Replacing consulo.ide.impl.idea.openapi.vfs.VfsUtil with consulo.virtualFileSystem.util.VirtualFileUtil. Replacing consulo.ide.impl.idea.openapi.util.io.FileUtil with consulo.util.io.FileUtil #293
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -54,7 +54,6 @@ | |
| import consulo.project.startup.StartupManager; | ||
| import consulo.util.lang.Pair; | ||
| import consulo.document.util.TextRange; | ||
| import consulo.ide.impl.idea.openapi.util.io.FileUtil; | ||
| import com.intellij.openapi.vfs.*; | ||
| import consulo.language.editor.inspection.scheme.InspectionProfileManager; | ||
|
Comment on lines
54
to
58
|
||
| import consulo.language.editor.inspection.scheme.InspectionProjectProfileManager; | ||
|
|
@@ -81,6 +80,8 @@ | |
| import consulo.ui.UIAccess; | ||
| import consulo.util.collection.primitive.ints.IntList; | ||
| import consulo.util.collection.primitive.ints.IntLists; | ||
| import consulo.util.io.FileUtil; | ||
| import consulo.virtualFileSystem.util.VirtualFileUtil; | ||
| import consulo.virtualFileSystem.LocalFileSystem; | ||
| import org.junit.Assert; | ||
|
|
||
|
|
@@ -541,7 +542,7 @@ protected void run(Result<PsiClass> result) throws Throwable | |
| File dir; | ||
| if(files.length > 0) | ||
| { | ||
| dir = VfsUtilCore.virtualToIoFile(files[0]); | ||
| dir = VirtualFileUtil.virtualToIoFile(files[0]); | ||
| } | ||
| else | ||
| { | ||
|
|
@@ -554,7 +555,7 @@ protected void run(Result<PsiClass> result) throws Throwable | |
| FileUtil.createIfDoesntExist(file); | ||
| VirtualFile vFile = LocalFileSystem.getInstance().refreshAndFindFileByPath(file.getCanonicalPath().replace(File.separatorChar, '/')); | ||
| Assert.assertNotNull(vFile); | ||
| consulo.ide.impl.idea.openapi.vfs.VfsUtil.saveText(vFile, text); | ||
| VirtualFileUtil.saveText(vFile, text); | ||
| PsiJavaFile psiFile = (PsiJavaFile) myPsiManager.findFile(vFile); | ||
| Assert.assertNotNull(psiFile); | ||
| PsiClass psiClass = psiFile.getClasses()[0]; | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -40,9 +40,9 @@ | |||||||
| import consulo.codeEditor.Editor; | ||||||||
| import consulo.project.Project; | ||||||||
| import consulo.util.lang.Pair; | ||||||||
| import consulo.ide.impl.idea.openapi.util.io.FileUtil; | ||||||||
| import consulo.util.lang.StringUtil; | ||||||||
| import consulo.util.io.CharsetToolkit; | ||||||||
|
||||||||
| import consulo.util.io.CharsetToolkit; | |
| import consulo.util.io.CharsetToolkit; | |
| import consulo.util.io.FileUtil; |
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -27,8 +27,8 @@ | |||||||
| import consulo.language.editor.WriteCommandAction; | ||||||||
| import consulo.module.Module; | ||||||||
| import consulo.module.ModuleManager; | ||||||||
|
||||||||
| import consulo.module.ModuleManager; | |
| import consulo.module.ModuleManager; | |
| import consulo.util.io.FileUtil; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,10 +25,10 @@ | |
| import consulo.ide.impl.idea.openapi.roots.ModuleRootModificationUtil; | ||
| import consulo.content.OrderRootType; | ||
| import consulo.util.xml.serializer.DefaultJDOMExternalizer; | ||
| import consulo.util.io.FileUtil; | ||
| import consulo.util.jdom.JDOMUtil; | ||
| import consulo.ide.impl.idea.openapi.util.io.FileUtil; | ||
| import consulo.util.lang.StringUtil; | ||
| import consulo.ide.impl.idea.openapi.vfs.VfsUtil; | ||
| import consulo.virtualFileSystem.util.VirtualFileUtil; | ||
| import consulo.virtualFileSystem.VirtualFile; | ||
| import com.intellij.psi.*; | ||
|
Comment on lines
27
to
33
|
||
| import consulo.language.impl.internal.psi.PsiManagerImpl; | ||
|
|
@@ -98,7 +98,7 @@ protected PsiFile createFile(final Module module, final VirtualFile vDir, final | |
| } | ||
|
|
||
| final VirtualFile vFile = vDir.createChildData(vDir, fileName); | ||
| VfsUtil.saveText(vFile, text); | ||
| VirtualFileUtil.saveText(vFile, text); | ||
| Assert.assertNotNull(vFile); | ||
| final PsiFile file = myPsiManager.findFile(vFile); | ||
| Assert.assertNotNull(file); | ||
|
|
@@ -116,7 +116,7 @@ protected PsiElement configureByFileWithMarker(String filePath, String marker) t | |
| final VirtualFile vFile = LocalFileSystem.getInstance().findFileByPath(filePath.replace(File.separatorChar, '/')); | ||
| Assert.assertNotNull("file " + filePath + " not found", vFile); | ||
|
|
||
| String fileText = VfsUtil.loadText(vFile); | ||
| String fileText = VirtualFileUtil.loadText(vFile); | ||
| fileText = StringUtil.convertLineSeparators(fileText); | ||
|
|
||
| int offset = fileText.indexOf(marker); | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After removing the old
FileUtil/VfsUtilimports, this class still usesFileUtil(e.g.,createTempFile,toSystemIndependentName) and now usesVirtualFileUtil.saveText/loadText, but neitherconsulo.util.io.FileUtilnorconsulo.virtualFileSystem.util.VirtualFileUtilare imported. Add the missing imports (or qualify the references) so the file compiles.