Chore: minor code cleanup for readability and functions return boolean (True/False) where needed for unittesting.

Author: xInsanityInRed

Travel_Itinerary_Planner/main.py

@@ -313,11 +313,13 @@ def run_app():
                             else:
                                 for flight in itinerary["flights"]:
                                     flight_options.append(flight["flight name"])
+                                    
                     # Checks if 'while True' statement should be broken
                     if not multiple_flights:
                         print("Returning to main menu...")
                         break
                     flight_id, itinerary_index = pick(flight_options, flight_prompt)
+
                     if delete_itinerary_item(itineraries, selected_type, itinerary_option, flight_id):
                         print(f"Flight '{flight_id}' has been deleted.")
                     else:
@@ -351,6 +353,7 @@ def run_app():
                         print("Returning to main menu...")
                         break
                     attraction_id, itinerary_index = pick(attraction_options, attraction_prompt)
+
                     if delete_itinerary_item(itineraries, selected_type, itinerary_option, attraction_id):
                         print(f"Attraction '{attraction_id}' has been deleted.")
                     else:

Travel_Itinerary_Planner/src/manage_itineraries.py

@@ -168,6 +168,7 @@ def add_new_flight(itinerary_list, itinerary_name, new_flights):
                 else:
                     print(f"Duplicate flight detected: {flight["flight name"]}!")
                     print("This flight will not be added.")
+                    return False
     save_itineraries(itinerary_list)
     # Uncomment to print for itinerary_list validation:
     # print(itinerary_list)
@@ -183,6 +184,7 @@ def add_new_attraction(itinerary_list, itinerary_name, new_attractions):
                 else:
                     print(f"Duplicate attraction detected: {attraction["attraction name"]}!")
                     print("This attraction will not be added.")
+                    return False
     save_itineraries(itinerary_list)
     # Uncomment to print for itinerary_list validation:
     # print(itinerary_list)
@@ -239,12 +241,12 @@ def print_table(trips):
     trip_table = Table(title="Itineraries", show_lines=True)
 
     trip_table.add_column("Trip Name", justify="center", no_wrap=True)
-    trip_table.add_column("Location", justify="center")
-    trip_table.add_column("Description", justify="center")
+    trip_table.add_column("Location", justify="center", no_wrap=True)
+    trip_table.add_column("Description", justify="center", no_wrap=False)
     trip_table.add_column("Start Date", justify="center", no_wrap=True)
     trip_table.add_column("End Date", justify="center", no_wrap=True)
     trip_table.add_column("Flights", justify="left", no_wrap=True)
-    trip_table.add_column("Attractions", justify="left", style="bold")
+    trip_table.add_column("Attractions", justify="left", style="bold", no_wrap=False)
 
     for trip in trips:
         flight_list = ''