Fix double form submit when using ENTER (fixes #72)

public/viewjs/batteryform.js

@@ -39,9 +39,10 @@ $('#battery-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('battery-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/batterytracking.js

@@ -68,9 +68,10 @@ $('#batterytracking-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('batterytracking-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/choreform.js

@@ -39,9 +39,10 @@ $('#chore-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+
 		if (document.getElementById('chore-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/choretracking.js

@@ -65,9 +65,10 @@ $('#choretracking-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+
 		if (document.getElementById('choretracking-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/consume.js

@@ -90,9 +90,10 @@ $('#consume-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('consume-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/inventory.js

@@ -118,9 +118,10 @@ $('#inventory-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('inventory-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/locationform.js

@@ -39,9 +39,10 @@ $('#location-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('location-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/productform.js

@@ -99,9 +99,10 @@ $('#product-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('product-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/productgroupform.js

@@ -39,9 +39,10 @@ $('#product-group-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('product-group-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/purchase.js

@@ -144,9 +144,10 @@ $('#purchase-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('purchase-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/quantityunitform.js

@@ -39,9 +39,10 @@ $('#quantityunit-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('quantityunit-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/recipeform.js

@@ -58,9 +58,10 @@ $('#recipe-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('recipe-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/recipeposform.js

@@ -88,9 +88,10 @@ $('#recipe-pos-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('recipe-pos-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/shoppinglistform.js

@@ -85,9 +85,10 @@ $('#shoppinglist-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('shoppinglist-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/taskcategoryform.js

@@ -39,9 +39,10 @@ $('#task-category-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('task-category-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/taskform.js

@@ -44,9 +44,10 @@ $('#task-form input').keydown(function(event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('task-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

public/viewjs/userform.js

@@ -49,9 +49,10 @@ $('#user-form input').keydown(function (event)
 {
 	if (event.keyCode === 13) //Enter
 	{
+		event.preventDefault();
+		
 		if (document.getElementById('user-form').checkValidity() === false) //There is at least one validation error
 		{
-			event.preventDefault();
 			return false;
 		}
 		else

views/batteryform.blade.php

@@ -47,7 +47,7 @@
 				'invalidFeedback' => $L('This cannot be negative')
 			))
 
-			<button id="save-battery-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-battery-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/batterytracking.blade.php

@@ -32,7 +32,7 @@
 				'invalidFeedback' => $L('This can only be before now')
 			))
 
-			<button id="save-batterytracking-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="save-batterytracking-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/choreform.blade.php

@@ -53,7 +53,7 @@
 				'additionalHtmlElements' => '<p id="chore-period-type-info" class="form-text text-muted small d-none"></p>'
 			))
 
-			<button id="save-chore-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-chore-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/choretracking.blade.php

@@ -39,7 +39,7 @@
 				'prefillByUserId' => GROCY_USER_ID
 			))
 
-			<button id="save-choretracking-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="save-choretracking-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/consume.blade.php

@@ -32,7 +32,7 @@
 				</label>
 			</div>
 
-			<button id="save-consume-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="save-consume-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/inventory.blade.php

@@ -42,7 +42,7 @@
 				'shortcutLabel' => 'Never expires'
 			))
 
-			<button id="save-inventory-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="save-inventory-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/locationform.blade.php

@@ -32,7 +32,7 @@
 				<textarea class="form-control" rows="2" id="description" name="description">@if($mode == 'edit'){{ $location->description }}@endif</textarea>
 			</div>
 
-			<button id="save-location-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-location-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/login.blade.php

@@ -29,7 +29,7 @@
 				</label>
 			</div>
 
-			<button id="login-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="login-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/productform.blade.php

@@ -108,7 +108,7 @@
 				'additionalHtmlElements' => '<p id="qu-conversion-info" class="form-text text-muted small d-none"></p>'
 			))
 
-			<button id="save-product-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-product-button" class="btn btn-success">{{ $L('Save') }}</button>
 		</form>
 	</div>
 </div>

views/productgroupform.blade.php

@@ -32,7 +32,7 @@
 				<textarea class="form-control" rows="2" id="description" name="description">@if($mode == 'edit'){{ $group->description }}@endif</textarea>
 			</div>
 
-			<button id="save-product-group-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-product-group-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/purchase.blade.php

@@ -49,7 +49,7 @@
 				'isRequired' => false
 			))
 
-			<button id="save-purchase-button" type="submit" class="btn btn-success">{{ $L('OK') }}</button>
+			<button id="save-purchase-button" class="btn btn-success">{{ $L('OK') }}</button>
 
 		</form>
 	</div>

views/quantityunitform.blade.php

@@ -37,7 +37,7 @@
 				<textarea class="form-control" rows="2" id="description" name="description">@if($mode == 'edit'){{ $quantityunit->description }}@endif</textarea>
 			</div>
 
-			<button id="save-quantityunit-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-quantityunit-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/recipeform.blade.php

@@ -36,7 +36,7 @@
 				<textarea id="description" class="form-control" name="description" rows="25">@if($mode == 'edit'){{ $recipe->description }}@endif</textarea>
 			</div>
 
-			<button id="save-recipe-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-recipe-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/recipeposform.blade.php

@@ -73,7 +73,7 @@
 				<textarea class="form-control" rows="2" id="note" name="note">@if($mode == 'edit'){{ $recipePos->note }}@endif</textarea>
 			</div>
 
-			<button id="save-recipe-pos-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-recipe-pos-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/shoppinglistform.blade.php

@@ -42,7 +42,7 @@
 				<textarea class="form-control" rows="2" id="note" name="note">@if($mode == 'edit'){{ $listItem->note }}@endif</textarea>
 			</div>
 
-			<button id="save-shoppinglist-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-shoppinglist-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/taskcategoryform.blade.php

@@ -32,7 +32,7 @@
 				<textarea class="form-control" rows="2" id="description" name="description">@if($mode == 'edit'){{ $category->description }}@endif</textarea>
 			</div>
 
-			<button id="save-task-category-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-task-category-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/taskform.blade.php

@@ -76,7 +76,7 @@
 				'prefillByUserId' => $initUserId
 			))
 
-			<button id="save-task-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-task-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>

views/userform.blade.php

@@ -48,7 +48,7 @@
 				<div class="invalid-feedback">{{ $L('Passwords do not match') }}</div>
 			</div>
 
-			<button id="save-user-button" type="submit" class="btn btn-success">{{ $L('Save') }}</button>
+			<button id="save-user-button" class="btn btn-success">{{ $L('Save') }}</button>
 
 		</form>
 	</div>