refresh in ABE options makes rulesets disappear

[al_9x]

NS 2.1.0.1, Fx 3.6.16, new profile

1. open abe options
2. select USER
3. click refresh, then ok
4. reopen abe options, the rulesets listbox is empty
5. PS 1: NS options will close with an ABE syntax error, that had not been highlighted, prior to clicking ok. I think this was addressed once.
6. PS 2: I think that for a large set of rules, file based storage is more user friendly than a huge pref string, and possibly more reliable

[al_9x]

I was clicking refresh just to force a syntax check, what is it actually for?

[Giorgio Maone]

Fixed in latest development build 2.1.0.2rc1.

# PS 1: NS options will close with an ABE syntax error, that had not been highlighted, prior to clicking ok. I think this was addressed once.

Not sure I understand, could you elaborate?

# PS 2: I think that for a large set of rules, file based storage is more user friendly than a huge pref string, and possibly more reliable

User friendly maybe, even though you can always copy from the textarea, edit wherever you want and paste back.
More reliable I doubt, and anyway all the other settings are there in preferences.
Surely heavier on startup performance, due to the additional I/O, and a pain to keep in sync without native file change notifications.

I was clicking refresh just to force a syntax check, what is it actually for?

It clears the cache of remote site rulesets, so when you visit them again they're refreshed.
It used also to reload the local ruleset files, but it doesn't affect local rulesets anymore because they're refreshed in real time thanks to pref changes notifications.

[al_9x]

# PS 1: NS options will close with an ABE syntax error, that had not been highlighted, prior to clicking ok. I think this was addressed once.

Not sure I understand, could you elaborate?

If you introduce a syntax error into an ABE ruleset and then when done editing, click ok to close the dialog, it closes, saving the error. The error highlight only briefly flashes, prior to closing, that's not an adequate notification. It should alert you about the error and stay open.

[al_9x]

More reliable I doubt

here's what i had in mind: when the rulsets disappeared, I went into about:config to check the abe prefs and remove my latest changes (thinking they were the cause). Modifying the pref via about:config stripped all the new lines, necessitating a manual reconstruction. All this just gave me an impression of flakiness of the new storage.

[dhouwn]

Modifying the pref via about:config stripped all the new lines, necessitating a manual reconstruction.

This is an about:config issue then: https://bugzilla.mozilla.org/show_bug.cgi?id=358984 /edit: Oops, that's not an UI bug report.

[Giorgio Maone]

It should alert you about the error and stay open.

I agree. Will be done before next release.

[Giorgio Maone]

It should alert you about the error and stay open.

I agree. Will be done before next release.

Done in latest development build.