Skip to content

BF: Avoid "untitled.psyexp does not exist" warning#6992

Merged
TEParsons merged 1 commit intopsychopy:devfrom
TEParsons:dev-bf-untitled-not-found
Nov 29, 2024
Merged

BF: Avoid "untitled.psyexp does not exist" warning#6992
TEParsons merged 1 commit intopsychopy:devfrom
TEParsons:dev-bf-untitled-not-found

Conversation

@TEParsons
Copy link
Copy Markdown
Contributor

If you closed Builder with nothing open (filename=untitled.psyexp, fileExists=False) then the filename still got added to prevFiles in app data, so when you next opened Builder it would try to open a nonexistant "untitled.psyexp" file and print a warning when it failed. The solution is to not store the filename if fileExists=False.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.83%. Comparing base (0b04eee) to head (ab92115).
Report is 6 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #6992   +/-   ##
=======================================
  Coverage   50.82%   50.83%           
=======================================
  Files         340      340           
  Lines       62318    62334   +16     
=======================================
+ Hits        31676    31688   +12     
- Misses      30642    30646    +4     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons TEParsons merged commit b97206f into psychopy:dev Nov 29, 2024
@TEParsons TEParsons deleted the dev-bf-untitled-not-found branch December 16, 2024 11:02
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.

1 participant