Skip to content

Commit

Permalink
use FileUtil.createTempFile() instead of File.createTempFile()
Browse files Browse the repository at this point in the history
  • Loading branch information
@labkey-matthewb
labkey-matthewb committed Dec 2, 2022
1 parent f2392b8 commit fdd07e5
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .idea/inspectionProfiles/Project_Default.xml
View file

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 comments on commit fdd07e5

@labkey-jeckels
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't highlighting for me. This does:

      <replaceConfiguration name="File.createTempFile() typically creates file with wide read permissions" text="java.io.File.createTempFile($Arg1$, $Arg2$)" recursive="false" caseInsensitive="true" type="JAVA" pattern_context="default" reformatAccordingToStyle="true" shortenFQN="true" replacement="org.labkey.api.util.FileUtil.createTempFile($Arg1$, $Arg2$)">
        <constraint name="__context__" within="" contains="" />
        <constraint name="Arg2" within="" contains="" />
        <constraint name="Arg1" within="" contains="" />
      </replaceConfiguration>

@mbellew
Copy link

@mbellew mbellew commented on fdd07e5 Dec 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With mine I got a highlight instead of the like I expected (but the replace worked). Does this work with the three parameter overload?

@labkey-jeckels
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, mine doesn't handle the three param version. Is there a replacement we should suggest for it?

Please sign in to comment.