search

omadawn / njord

Java iRule Editor
2 stars 1 forks source link

Update 'New iRule' to handle the new way that the jTree is built (with textEditorPanes) #39

Closed omadawn closed 12 years ago

omadawn commented 12 years ago

New iRule is completely broken now.

omadawn commented 12 years ago

Started to create buildiRuleListOfEditors(List iRules) and making NjordiRuleObject hold a TextEditorPane but I'm no longer sure I actually want that.

omadawn commented 12 years ago

I might not need to open the editor with a fileLocaion if I create an editor and then use setText() the editorpane will have a 'local' flag boolean thisIsLocal = ruleEditor.isLocal();

I can use that to determine if I need to create it or save it.

If I create one on the server I will need to commpletely rebuild the nav tree. Which actually is functionality I'm trying to add anyway.

omadawn commented 12 years ago

I also need to check the name of the rule against those on the server and throw an error if it conflicts.

omadawn commented 12 years ago

Updated new irule functionality so that it creates the editor and adds it to the tree. That totally screwed up the gui. Handled that issue under Issue #68

omadawn commented 12 years ago

Fixed that and then having an issue where we weren't erroring out if the iRule already existed. Fixed that.

omadawn commented 12 years ago

Now having a problem where if you save the iRule it doesn't actually save it. There's the other issue where the TextEditorPane is being marked clean when it shouldn't be but i want to address the issue where it doesn't actually save here in this issue.

omadawn commented 12 years ago

Woohooo it works now! yay!

Got it to save but then subsequent saves would fail.

Fixed that part too by setting the fileLocation to no longer be local.