Index: lams_central/web/lams_authoring.swf
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/lams_authoring.swf,v
diff -u -r1.495 -r1.496
Binary files differ
Index: lams_central/web/lams_authoring_library.swf
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/lams_authoring_library.swf,v
diff -u -r1.395 -r1.396
Binary files differ
Index: lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceContainer.as
===================================================================
RCS file: /usr/local/cvsroot/lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceContainer.as,v
diff -u -r1.5 -r1.6
--- lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceContainer.as	4 Nov 2008 01:22:10 -0000	1.5
+++ lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceContainer.as	17 Nov 2008 22:48:13 -0000	1.6
@@ -63,7 +63,6 @@
 	private var delete_competence_btn:Button;
 	
     private var _bgpanel:MovieClip;       //The underlaying panel base
-	private var definitionDialog:MovieClip;
 	
 	private var app:Application;
     private var fm:FocusManager;            //Reference to focus manager
@@ -152,10 +151,9 @@
 	
 	public function editHandler():Void {
 		Debugger.log("editCompetence clicked: _competenceTitle "+_competenceTitle, Debugger.CRITICAL, "editHandler", "CompetenceContainer");
-		CompetenceEditorDialog(app.dialog.scrollContent).editingCompetence = _competenceTitle;
-		
-		definitionDialog = app.getCanvas().openCompetenceDefinitionWindow();
-		Selection.setFocus(definitionDialog.scrollContent);
+		CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).editingCompetence = _competenceTitle;
+		app.getCanvas().openCompetenceDefinitionWindow(); // sets app.dialog to the definition dialog
+		Selection.setFocus(app.dialog.scrollContent);
 	}
 	
 	public function deleteHandler():Void {
@@ -215,8 +213,7 @@
 	
 	
 	private function deleteCompetence() {
-		// app.dialog.scrollContent must be an instance of CompetenceEditorDialog
-		var competenceEditorDialog:CompetenceEditorDialog = CompetenceEditorDialog(app.dialog.scrollContent);
+		var competenceEditorDialog:CompetenceEditorDialog = CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent);
 		app.getDesignDataModel().competences.remove(_competenceTitle);
 			
 		competenceEditorDialog.clear();
Index: lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceDefinitionDialog.as
===================================================================
RCS file: /usr/local/cvsroot/lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceDefinitionDialog.as,v
diff -u -r1.6 -r1.7
--- lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceDefinitionDialog.as	4 Nov 2008 00:01:17 -0000	1.6
+++ lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceDefinitionDialog.as	17 Nov 2008 22:48:13 -0000	1.7
@@ -114,7 +114,7 @@
 		add_competence_btn.label = Dictionary.getValue("competence_editor_add_competence_btn");
 		save_competence_btn.label = Dictionary.getValue("mnu_file_save");
 		
-		editingCompetence = CompetenceEditorDialog(app.dialog.scrollContent).editingCompetence;
+		editingCompetence = CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).editingCompetence;
 		
         //EVENTS
         //Add event listeners for ok, cancel and close buttons
@@ -210,21 +210,20 @@
 		var competenceDesc:String = StringUtils.trim(competence_description_txt.text);
 		
 		if (competenceName.length == 0) {
-			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_blank"), null);
+			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_blank"), Delegate.create(this, selectAndHighlightTitle));
 		}
 		else if (app.getDesignDataModel().competences.containsKey(competenceName)) {
 			// entry already exists, do not add
-			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_exists", [competenceName]), null);
+			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_exists", [competenceName]), Delegate.create(this, selectAndHighlightTitle));
 		} 
 		else {
 			// add the new competence to the hashtable
 			app.getDesignDataModel().competences.put(competenceName, competenceDesc);
 
-			CompetenceEditorDialog(app.dialog.scrollContent).clear();
-			CompetenceEditorDialog(app.dialog.scrollContent).draw();
+			CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).clear();
+			CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).draw();
+			_container.deletePopUp();
 		}
-		
-		_container.deletePopUp();
 	}
 	
 	// used for updating an existing competence
@@ -234,54 +233,52 @@
 		var updatedCompetenceDesc:String = StringUtils.trim(competence_description_txt.text);
 		
 		if (updatedCompetenceName.length == 0) {
-			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_blank"), null);
+			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_blank"), Delegate.create(this, selectAndHighlightTitle));
 		}
 		else if (editingCompetence != updatedCompetenceName && app.getDesignDataModel().competences.containsKey(updatedCompetenceName)) {
 			// entry already exists, do not add
-			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_exists", [updatedCompetenceName]), null);
+			LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_exists", [updatedCompetenceName]), Delegate.create(this, selectAndHighlightTitle));
 		} 
 		else {
-		
 			if (editingCompetence != null && editingCompetence != undefined && app.getDesignDataModel().competences.containsKey(editingCompetence)) {
-				if (updatedCompetenceName.length > 0) {
-					app.getDesignDataModel().competences.remove(editingCompetence);  // remove the original competencetitle
-					app.getDesignDataModel().competences.put(updatedCompetenceName, updatedCompetenceDesc);
+				
+				app.getDesignDataModel().competences.remove(editingCompetence);  // remove the original competencetitle
+				app.getDesignDataModel().competences.put(updatedCompetenceName, updatedCompetenceDesc);
 		
-					// if the competence title has changed, update the mappings for all activities to the new competence title
-					if (editingCompetence != updatedCompetenceName) { 
+				// if the competence title has changed, update the mappings for all activities to the new competence title
+				if (editingCompetence != updatedCompetenceName) { 
 						
-						var activityKeys:Array = app.getDesignDataModel().activities.keys(); // contains the activity UIIDs
+					var activityKeys:Array = app.getDesignDataModel().activities.keys(); // contains the activity UIIDs
 						
-						for (var i=0; i<activityKeys.length; ++i) {
+					for (var i=0; i<activityKeys.length; ++i) {
 
-							if (app.getDesignDataModel().activities.get(activityKeys[i]) instanceof ToolActivity) {
-								var competenceMappings:Array = ToolActivity(app.getDesignDataModel().activities.get(activityKeys[i])).competenceMappings;
-								for (var j=0; j<competenceMappings.length; ++j) {
-									if (competenceMappings[j] == editingCompetence) {
-										competenceMappings[j] = updatedCompetenceName;
-									}
+						if (app.getDesignDataModel().activities.get(activityKeys[i]) instanceof ToolActivity) {
+							var competenceMappings:Array = ToolActivity(app.getDesignDataModel().activities.get(activityKeys[i])).competenceMappings;
+							for (var j=0; j<competenceMappings.length; ++j) {
+								if (competenceMappings[j] == editingCompetence) {
+									competenceMappings[j] = updatedCompetenceName;
 								}
 							}
 						}
 					}
 					
-					CompetenceEditorDialog(app.dialog.scrollContent).clear();
-					CompetenceEditorDialog(app.dialog.scrollContent).draw();
+					CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).clear();
+					CompetenceEditorDialog(app.canvas.model.competenceEditorDialog.scrollContent).draw();
 				}
-				else {
-					LFMessage.showMessageAlert(Dictionary.getValue("competence_editor_warning_title_blank"), null);
-				}
+				_container.deletePopUp();
 			}
 		}
-		
-		_container.deletePopUp();
 	}
 	
+	private function selectAndHighlightTitle():Void {
+		Selection.setFocus(competence_title_txt);
+	}
+	
     /**
     * Called by the OK button 
     */
     private function close(){
-
+		app.dialog = app.canvas.model.competenceEditorDialog;
 		_container.deletePopUp();
 
     }
@@ -290,6 +287,7 @@
     * Event dispatched by parent container when close button clicked
     */
     public function click(e:Object):Void{
+		app.dialog = app.canvas.model.competenceEditorDialog;
         e.target.deletePopUp();
     }
     
Index: lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceEditorDialog.as
===================================================================
RCS file: /usr/local/cvsroot/lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceEditorDialog.as,v
diff -u -r1.4 -r1.5
--- lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceEditorDialog.as	8 Oct 2008 00:13:27 -0000	1.4
+++ lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cmpt/CompetenceEditorDialog.as	17 Nov 2008 22:48:13 -0000	1.5
@@ -62,7 +62,6 @@
 	private var competenceContainer:MovieClip;
 	private var main_mc:MovieClip
 	private var competenceContainerLayer:MovieClip;
-	private var definitionDialog:MovieClip;
 	
 	private var containerArray:Array;
 	
@@ -233,8 +232,8 @@
 		
 		_editingCompetence = null;
 		
-		definitionDialog = app.getCanvas().openCompetenceDefinitionWindow();
-		Selection.setFocus(definitionDialog.scrollContent);
+		app.getCanvas().openCompetenceDefinitionWindow(); // sets app.dialog to the definition window
+		Selection.setFocus(app.dialog.scrollContent);
 	}
 
 	/**
@@ -278,6 +277,8 @@
     * Called by the close button 
     */
     private function close(){
+		if (model instanceof CanvasModel)
+			app.canvas.model.competenceEditorDialog = null;
 		
 		_container.deletePopUp();
     }
@@ -286,7 +287,10 @@
     * Event dispatched by parent container when close button clicked
     */
     public function click(e:Object):Void{
-        e.target.deletePopUp();
+		if (model instanceof CanvasModel)
+			app.canvas.model.competenceEditorDialog = null;
+        
+		e.target.deletePopUp();
     }
 	
 	/**
Index: lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasHelper.as
===================================================================
RCS file: /usr/local/cvsroot/lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasHelper.as,v
diff -u -r1.46 -r1.47
--- lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasHelper.as	14 Nov 2008 04:29:47 -0000	1.46
+++ lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasHelper.as	17 Nov 2008 22:48:13 -0000	1.47
@@ -79,7 +79,7 @@
 	private static var AUTOSAVE_CONFIG:String = "autosave";
 	private static var AUTOSAVE_TAG:String = "cv.ddm.autosave.user.";
    
-   private var _bin:MovieClip;
+	private var _bin:MovieClip;
 	
 	private var _target_mc:MovieClip;
 	
@@ -826,17 +826,18 @@
 
 		app.dialog = PopUpManager.createPopUp(Application.root, LFWindow, true,{title:Dictionary.getValue('competence_editor_dlg'), closeButton:true, resize:false, scrollContentPath:'CompetenceEditorDialog'});
 		app.dialog.addEventListener('contentLoaded',Delegate.create(controller, controller.openDialogLoaded));
+		
+		canvasModel.competenceEditorDialog = app.dialog; // store a reference to the competence editor dialog
 	}	
 	
 	/**
     * Opens the competence definition dialog
     */
-    public function openCompetenceDefinitionWindow():MovieClip {
+    public function openCompetenceDefinitionWindow() {
 		var controller:CanvasController = canvasView.getController();
-		var dialog:MovieClip = PopUpManager.createPopUp(Application.root, LFWindow, true,{title:Dictionary.getValue('competence_def_dlg'), closeButton:true, scrollContentPath:'CompetenceDefinitionDialog'});			
-		
-		dialog.addEventListener('contentLoaded',Delegate.create(controller, controller.openDialogLoaded));
-		return dialog;
+	
+		app.dialog = PopUpManager.createPopUp(Application.root, LFWindow, true,{title:Dictionary.getValue('competence_def_dlg'), closeButton:true, scrollContentPath:'CompetenceDefinitionDialog'});	
+		app.dialog.addEventListener('contentLoaded',Delegate.create(controller, controller.openDialogLoaded));
 	}
 	
 	public function openBranchView(ba, visible:Boolean){
Index: lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasSuperModel.as
===================================================================
RCS file: /usr/local/cvsroot/lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasSuperModel.as,v
diff -u -r1.37 -r1.38
--- lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasSuperModel.as	29 Oct 2008 04:16:20 -0000	1.37
+++ lams_flash/src/central/flash/org/lamsfoundation/lams/authoring/cv/CanvasSuperModel.as	17 Nov 2008 22:48:13 -0000	1.38
@@ -81,6 +81,8 @@
 	private var _openBranchingActivities:Array;
 	private var _openBranchingActivity:Number;
 	
+	private var _competenceEditorDialog:MovieClip;
+	
 	//These are defined so that the compiler can 'see' the events that are added at runtime by EventDispatcher
     private var dispatchEvent:Function;     
     public var addEventListener:Function;
@@ -1192,6 +1194,14 @@
 		return _openBranchingActivity;
 	}
 	
+	public function set competenceEditorDialog(dialog:MovieClip) {
+		_competenceEditorDialog = dialog;
+	}
+	
+	public function get competenceEditorDialog():MovieClip {
+		return _competenceEditorDialog;
+	}
+	
 	public function get openBranchingActivities():Array {
 		return _openBranchingActivities;
 	}