inventree/inventree-app
2
0
Fork 0
mirror of https://github.com/inventree/inventree-app.git synced 2025-04-29 05:56:47 +00:00
Code

Cleanup of action dialogs

Browse Source
This commit is contained in:
Oliver Walters 2020-04-10 00:19:01 +10:00
parent 5a9dead4e6
commit 6799e9c85b
1 changed files with 19 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
lib/widget/stock_detail.dart
View File

@ -27,7 +27,7 @@ class StockDetailWidget extends StatefulWidget {
class _StockItemDisplayState extends State<StockDetailWidget> { class _StockItemDisplayState extends State<StockDetailWidget> {
// Single TextEditingController which can be shared between dialogs final TextEditingController _quantityController = TextEditingController();
final TextEditingController _notesController = TextEditingController(); final TextEditingController _notesController = TextEditingController();
final _addStockKey = GlobalKey<FormState>(); final _addStockKey = GlobalKey<FormState>();
@ -49,6 +49,9 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
Navigator.of(context).pop(); Navigator.of(context).pop();
double quantity = double.parse(_quantityController.text);
_quantityController.clear();
// Await response to prevent the button from being pressed multiple times // Await response to prevent the button from being pressed multiple times
var response = await item.addStock(quantity, notes: _notesController.text); var response = await item.addStock(quantity, notes: _notesController.text);
_notesController.clear(); _notesController.clear();
@ -70,7 +73,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
FlatButton( FlatButton(
child: Text("Add"), child: Text("Add"),
onPressed: () { onPressed: () {
_addStockKey.currentState.validate(); if (_addStockKey.currentState.validate()) _addStock();
}, },
) )
], ],
@ -87,6 +90,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
labelText: "Add stock", labelText: "Add stock",
), ),
keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true), keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true),
controller: _quantityController,
validator: (value) { validator: (value) {
if (value.isEmpty) return "Value cannot be empty"; if (value.isEmpty) return "Value cannot be empty";
@ -94,8 +98,6 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
if (quantity == null) return "Value cannot be converted to a number"; if (quantity == null) return "Value cannot be converted to a number";
if (quantity <= 0) return "Value must be positive"; if (quantity <= 0) return "Value must be positive";
_addStock(quantity);
return null; return null;
}, },
), ),
@ -114,9 +116,12 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
// TODO - Form for adding stock // TODO - Form for adding stock
} }
void _removeStock(double quantity) async { void _removeStock() async {
Navigator.of(context).pop(); Navigator.of(context).pop();
double quantity = double.parse(_quantityController.text);
_quantityController.clear();
var response = await item.removeStock(quantity, notes: _notesController.text); var response = await item.removeStock(quantity, notes: _notesController.text);
_notesController.clear(); _notesController.clear();
@ -136,7 +141,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
FlatButton( FlatButton(
child: Text("Remove"), child: Text("Remove"),
onPressed: () { onPressed: () {
_removeStockKey.currentState.validate(); if (_removeStockKey.currentState.validate()) _removeStock();
}, },
) )
], ],
@ -152,6 +157,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
decoration: InputDecoration( decoration: InputDecoration(
labelText: "Remove stock", labelText: "Remove stock",
), ),
controller: _quantityController,
keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true), keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true),
validator: (value) { validator: (value) {
if (value.isEmpty) return "Value cannot be empty"; if (value.isEmpty) return "Value cannot be empty";
@ -163,8 +169,6 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
if (quantity > item.quantity) return "Cannot take more than current quantity"; if (quantity > item.quantity) return "Cannot take more than current quantity";
_removeStock(quantity);
return null; return null;
}, },
), ),
@ -182,10 +186,13 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
); );
} }
void _countStock(double quantity) async { void _countStock() async {
Navigator.of(context).pop(); Navigator.of(context).pop();
double quantity = double.parse(_quantityController.text);
_quantityController.clear();
var response = await item.countStock(quantity, notes: _notesController.text); var response = await item.countStock(quantity, notes: _notesController.text);
_notesController.clear(); _notesController.clear();
@ -207,7 +214,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
FlatButton( FlatButton(
child: Text("Count"), child: Text("Count"),
onPressed: () { onPressed: () {
_countStockKey.currentState.validate(); if (_countStockKey.currentState.validate()) _countStock();
}, },
) )
], ],
@ -223,6 +230,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
labelText: "Count stock", labelText: "Count stock",
hintText: "${item.quantity}", hintText: "${item.quantity}",
), ),
controller: _quantityController,
keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true), keyboardType: TextInputType.numberWithOptions(signed: false, decimal: true),
validator: (value) { validator: (value) {
if (value.isEmpty) return "Value cannot be empty"; if (value.isEmpty) return "Value cannot be empty";
@ -231,8 +239,6 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
if (quantity == null) return "Value cannot be converted to a number"; if (quantity == null) return "Value cannot be converted to a number";
if (quantity < 0) return "Value cannot be negative"; if (quantity < 0) return "Value cannot be negative";
_countStock(quantity);
return null; return null;
}, },
), ),
@ -277,7 +283,7 @@ class _StockItemDisplayState extends State<StockDetailWidget> {
), ),
trailing: IconButton( trailing: IconButton(
icon: FaIcon(FontAwesomeIcons.edit), icon: FaIcon(FontAwesomeIcons.edit),
onPressed: _editStockItem, onPressed: _editStockItemDialog,
) )
) )
) )