diff --git a/gemp-lotr/gemp-lotr-web/src/main/webapp/includes/changeLog.html b/gemp-lotr/gemp-lotr-web/src/main/webapp/includes/changeLog.html index 668faaade..aa9eafd6b 100644 --- a/gemp-lotr/gemp-lotr-web/src/main/webapp/includes/changeLog.html +++ b/gemp-lotr/gemp-lotr-web/src/main/webapp/includes/changeLog.html @@ -1,6 +1,8 @@
 02 Jan. 2013
 - "Countless Companies" should now correctly be able to "catch" possessions.
+- When assignments are done, if there is at least one minion unassigned (that could have been assigned), player will
+have to confirm these assignments to proceed.
 
 01 Jan. 2013
 - "Scintillating Bird" now should discard the cards, if not enough cards of the specified type are revealed.
diff --git a/gemp-lotr/gemp-lotr-web/src/main/webapp/js/gemp-014/gameUi.js b/gemp-lotr/gemp-lotr-web/src/main/webapp/js/gemp-014/gameUi.js
index e62547d3f..f0a9105ef 100644
--- a/gemp-lotr/gemp-lotr-web/src/main/webapp/js/gemp-014/gameUi.js
+++ b/gemp-lotr/gemp-lotr-web/src/main/webapp/js/gemp-014/gameUi.js
@@ -49,8 +49,6 @@ var GempLotrGameUI = Class.extend({
     shadowAssignGroups:null,
     freePeopleAssignGroups:null,
 
-    assignmentStarted:null,
-
     selectionFunction:null,
 
     chatBoxDiv:null,
@@ -1870,22 +1868,13 @@ var GempLotrGameUI = Class.extend({
 
         var that = this;
 
-        this.assignmentStarted = false;
-
         this.alertText.html(text);
         // ****CCG League****: Border around alert box
         this.alertBox.css({"border-radius":"7px", "border-color":"#7faf7f", "border-width":"2px"});
         if (!this.replayMode) {
             this.alertButtons.html("");
             $("#Done").button().click(function () {
-                if (!that.assignmentStarted && !confirm("Do you want to skip assigning?"))
-                    return;
-
-                that.alertText.html("");
-                // ****CCG League****: Border around alert box
-                that.alertBox.css({"border-radius":"7px", "border-color":"", "border-width":"1px"});
-                that.alertButtons.html("");
-                that.clearSelection();
+                var atLeastOnMinionUnassigned = false;
 
                 var assignmentMap = {};
                 for (var i = 0; i < freeCharacters.length; i++) {
@@ -1896,13 +1885,24 @@ var GempLotrGameUI = Class.extend({
                         var card = $(this).data("card");
                         if (card.assign != null)
                             assignmentMap[card.assign] += " " + card.cardId;
+                        else
+                            atLeastOnMinionUnassigned=true;
                     });
 
+                if (atLeastOnMinionUnassigned && !confirm("At least one minion has not been assigned, do you want to proceed?"))
+                    return;
+
                 var assignmentArray = new Array();
                 for (var i = 0; i < freeCharacters.length; i++) {
                     assignmentArray.push(assignmentMap[freeCharacters[i]]);
                 }
 
+                that.alertText.html("");
+                // ****CCG League****: Border around alert box
+                that.alertBox.css({"border-radius":"7px", "border-color":"", "border-width":"1px"});
+                that.alertButtons.html("");
+                that.clearSelection();
+
                 that.decisionFunction(id, "" + assignmentArray);
             });
         }
@@ -1954,7 +1954,6 @@ var GempLotrGameUI = Class.extend({
     doAssignments:function (freeCharacters, minions) {
         var that = this;
         this.selectionFunction = function (cardId) {
-            that.assignmentStarted = true;
             that.clearSelection();
 
             that.selectionFunction = function (secondCardId) {