-
Notifications
You must be signed in to change notification settings - Fork 4
Description
I used to prepend Visualizer/ prefix to downloaded profiles, but then a while back removed it. So I was surprised when I pulled a certain shot and it was still prefixed. I triple checked my code and was so confused what was going on. And then I found this:
Lines 2345 to 2350 in 87676c5
set pparts [split $profile(profile_title) "/"] | |
if { [llength $pparts] == 1 } { | |
set profile(profile_title) "Visualizer/$profile(profile_title)" | |
} elseif { [lindex $pparts 1] ne "Visualizer" } { | |
set profile(profile_title) "Visualizer/[lindex $pparts end]" | |
} |
The reason I removed it was this discussion: https://3.basecamp.com/3671212/buckets/7351439/messages/7534772989#__recording_7836790146
I see that your code is now preventing multiple nesting, but still might be causing D-Flow issues. I'm not sure, since I don't use it, but maybe there are other issues?
Feel free to close this issue, if you don't think it's an issue. I just wanted to let you know that I've removed the prefixing and that you might follow suite. Or not. As you wish 😅