Fix: improve combo package selection UI and empty state handling
- add Select a Combo Package header - show message when no combo packages are available - improve combo package table header labels Fixes #1743
This commit is contained in:
@@ -216,8 +216,15 @@ Return type: void
|
|||||||
void CustomerMenu::selectComboPackage()
|
void CustomerMenu::selectComboPackage()
|
||||||
{
|
{
|
||||||
std::string vehicleNumber, vehicleBrand, vehicleModel;
|
std::string vehicleNumber, vehicleBrand, vehicleModel;
|
||||||
auto comboPackages = m_controller.getComboPackages();
|
|
||||||
util::clear();
|
util::clear();
|
||||||
|
std::cout << "Select a Combo Package\n";
|
||||||
|
auto comboPackages = m_controller.getComboPackages();
|
||||||
|
if (comboPackages.isEmpty())
|
||||||
|
{
|
||||||
|
std::cout << "No combo packages available!";
|
||||||
|
util::pressEnter();
|
||||||
|
return;
|
||||||
|
}
|
||||||
const ComboPackage* selectedComboPackage = selectComboPackageFromPackages(comboPackages);
|
const ComboPackage* selectedComboPackage = selectComboPackageFromPackages(comboPackages);
|
||||||
if (selectedComboPackage == nullptr)
|
if (selectedComboPackage == nullptr)
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -784,8 +784,8 @@ inline const ComboPackage* selectComboPackageFromPackages(const util::Map<std::s
|
|||||||
int userInputIndex;
|
int userInputIndex;
|
||||||
std::cout << std::left
|
std::cout << std::left
|
||||||
<< std::setw(10) << "Index"
|
<< std::setw(10) << "Index"
|
||||||
<< std::setw(15) << "Combo Package ID"
|
<< std::setw(15) << "Combo ID"
|
||||||
<< std::setw(15) << "Combo Package Name"
|
<< std::setw(15) << "Combo Name"
|
||||||
<< std::setw(15) << "Estimate Cost"
|
<< std::setw(15) << "Estimate Cost"
|
||||||
<< std::endl;
|
<< std::endl;
|
||||||
for (int index = 0; index < comboPackages.getSize(); index++)
|
for (int index = 0; index < comboPackages.getSize(); index++)
|
||||||
|
|||||||
Reference in New Issue
Block a user