[jg18]

Ok, builds updated with the fix. Var replacement is unchanged.

The changes from the last build began with commits on April 8: https://github.com/jg18/fs2open.github.com/commits/feature/sexp-containers-revised

[karajorma]

Here's a quick test mission I put together to show what values in the briefings and debriefs would be expected by a FREDder. As you can see they're not correct.

[attachment deleted to save space]

[jg18]

I created a PR that should fix this.

EDIT: removed link to test build (no longer needed since the fix is now in master).

[jg18]

Builds in the OP updated with minor fixes and cleanup.

Hoping to start a PR in the next few days.

The remaining TODOs can get sorted out as part of PR review.

[jg18]

Builds in the OP updated with (untested) container persistence support.

The only missing part is FRED controls to enable it. However, you can add the persistence options in Notepad, which should go after the $Data line.

The options (case-insensitive, include the '+'):

+Eternal
+Save On Mission Progress
+Save On Mission Close

"+Eternal", if used, must come before the "Save On" option.

Even though FRED doesn't have controls yet, it should still load/save the container persistence options.

EDIT: Forgot to mention that persistent containers should work for "Red Alert" missions as well.

[karajorma]

Awesome. Will download and test now.

[karajorma]

A couple of minor issues I just noticed while putting together a campaign to test persistence.

• If you want to rename a container and you open the dialog, rename it and click OK, it will be updated. If you rename it and then open a different variable, it isn't renamed on the list until the dialogue is reopened (which of course makes look like it hasn't been renamed)
• If you try to rename an existing list by adding a space (Say renaming List -> List 2) as soon as you type the space the dialog will open up to tell you that names can't end in a space. You can avoid the issue by typing in the name without spaces and then adding them but it's a little clunky.

[jg18]

Builds in OP updated with FRED controls for persistence options and also a formatting fix when saving persistence options to disk.

For the UI glitches with container renaming, I'm thinking of using a separate dialog to rename containers, something similar to the Add New Container dialog, and then making the list of container names non-editable. That'll make it much easier to fix those bugs.

[jg18]

Builds in OP updated.

The main change is that the drop down box of container names is no longer editable. Instead, there's a new "Rename Container" button.

The container names box is now disabled when the last container is deleted.

The FRED UI glitch about a renamed container's name not getting updated in the drop down box should now be fixed.

I also rebased against latest master, which now includes my fix for variable replacement.