Update menu.py

menu.py

@@ -1,6 +1,6 @@
 from flask import Blueprint, render_template, request, session, jsonify, redirect, url_for
-from salesforce import get_salesforce_connection
 import os
+from salesforce import get_salesforce_connection
 
 menu_blueprint = Blueprint('menu', __name__)
 
@@ -15,7 +15,8 @@ SECTION_ORDER = ["Best Sellers", "Starters", "Biryanis", "Curries", "Breads", "C
 
 # Create placeholder video at startup if it doesn't exist
 if not os.path.exists(PLACEHOLDER_PATH):
-    open(PLACEHOLDER_PATH, 'wb').close()
+    with open(PLACEHOLDER_PATH, 'wb') as f:
+        f.close()
     print(f"Created placeholder video at {PLACEHOLDER_PATH}")
 
 def get_valid_video_path(item_name, video_url=None):
@@ -33,11 +34,12 @@ def get_valid_video_path(item_name, video_url=None):
             return video_url
         # If it's a Salesforce File ID (starts with '069')
         elif video_url.startswith('069'):
-            return f"https://yourdomain.my.salesforce.com/sfc/servlet.shepherd/version/download/{video_url}"
+            return f"https://biryanihub-dev-ed.develop.my.salesforce.com/sfc/servlet.shepherd/version/download/{video_url}"
     
     # Final fallback: placeholder.mp4
     if not os.path.exists(PLACEHOLDER_PATH):
-        open(PLACEHOLDER_PATH, 'wb').close()
+        with open(PLACEHOLDER_PATH, 'wb') as f:
+            f.close()
         print(f"Created missing placeholder video at {PLACEHOLDER_PATH}")
     
     return f"/static/{PLACEHOLDER_VIDEO}"
@@ -47,11 +49,11 @@ def menu():
     selected_category = request.args.get("category", "All")
     user_email = session.get('user_email')
 
+    # Handle user authentication
     if not user_email:
         user_email = request.args.get("email")
         user_name = request.args.get("name")
-
-        if user_email:
+        if user_email and user_name:
             session['user_email'] = user_email
             session['user_name'] = user_name
         else:
@@ -65,8 +67,7 @@ def menu():
         # Fetch user referral and reward points
         user_query = f"SELECT Referral__c, Reward_Points__c FROM Customer_Login__c WHERE Email__c = '{user_email}'"
         user_result = sf.query(user_query)
-
-        if not user_result['records']:
+        if not user_result.get('records'):
             return redirect(url_for('login'))
 
         referral_code = user_result['records'][0].get('Referral__c', 'N/A')
@@ -75,96 +76,86 @@ def menu():
         # Get cart item count
         cart_query = f"SELECT COUNT() FROM Cart_Item__c WHERE Customer_Email__c = '{user_email}'"
         cart_count_result = sf.query(cart_query)
-        cart_item_count = cart_count_result['totalSize']
+        cart_item_count = cart_count_result.get('totalSize', 0)
 
-        # Query to fetch Menu_Item__c records including Video1__c and nutrition fields
+        # Fetch all Menu_Item__c records
         menu_query = """
             SELECT Name, Price__c, Description__c, Image1__c, Image2__c, 
-                   Veg_NonVeg__c, Section__c, Total_Ordered__c, Video1__c,
-                   IngredientsInfo__c, Nutritional_Info__c, Allergens__c
+                   Veg_NonVeg__c, Section__c, Total_Ordered__c, Video1__c
             FROM Menu_Item__c
         """
-        result = sf.query(menu_query)
-        food_items = result['records'] if 'records' in result else []
+        menu_result = sf.query_all(menu_query)  # Use query_all to fetch all records
+        food_items = menu_result.get('records', [])
 
-        # Process items and add video paths
+        # Process menu items and add video paths
         for item in food_items:
-            if 'Total_Ordered__c' not in item or item['Total_Ordered__c'] is None:
-                item['Total_Ordered__c'] = 0
+            item['Total_Ordered__c'] = item.get('Total_Ordered__c', 0) or 0
             item['Video1__c'] = get_valid_video_path(item['Name'], item.get('Video1__c'))
+            item['Section__c'] = item.get('Section__c') or "Others"  # Default section
 
-        # Query to fetch Custom_Dish__c records with nutrition fields
+        # Fetch all Custom_Dish__c records from the last 7 days
         custom_dish_query = """
             SELECT Name, Price__c, Description__c, Image1__c, Image2__c, 
-                   Veg_NonVeg__c, Section__c, Total_Ordered__c,
-                   IngredientsInfo__c, Nutritional_Info__c, Allergens__c
+                   Veg_NonVeg__c, Section__c, Total_Ordered__c
             FROM Custom_Dish__c
             WHERE CreatedDate >= LAST_N_DAYS:7
         """
-        custom_dish_result = sf.query(custom_dish_query)
+        custom_dish_result = sf.query_all(custom_dish_query)
         custom_dishes = custom_dish_result.get('records', [])
 
         # Process custom dishes and add video paths
         for item in custom_dishes:
-            if 'Total_Ordered__c' not in item or item['Total_Ordered__c'] is None:
-                item['Total_Ordered__c'] = 0
+            item['Total_Ordered__c'] = item.get('Total_Ordered__c', 0) or 0
             item['Video1__c'] = get_valid_video_path(item['Name'])
+            item['Section__c'] = item.get('Section__c') or "Customized dish"  # Default section
 
-        # Merge both Menu_Item__c and Custom_Dish__c records
+        # Merge all items
         all_items = food_items + custom_dishes
         ordered_menu = {section: [] for section in SECTION_ORDER}
 
-        # Process best sellers
-        best_sellers = sorted(all_items, key=lambda x: x.get("Total_Ordered__c", 0), reverse=True)
-        
+        # Process best sellers (top 4 by Total_Ordered__c)
+        best_sellers = sorted(all_items, key=lambda x: x['Total_Ordered__c'], reverse=True)
         if selected_category == "Veg":
             best_sellers = [item for item in best_sellers if item.get("Veg_NonVeg__c") in ["Veg", "both"]]
         elif selected_category == "Non veg":
             best_sellers = [item for item in best_sellers if item.get("Veg_NonVeg__c") in ["Non veg", "both"]]
+        ordered_menu["Best Sellers"] = best_sellers[:4]
 
-        best_sellers = best_sellers[:4]
-        if best_sellers:
-            ordered_menu["Best Sellers"] = best_sellers
-
-        # Organize other sections
-        added_item_names = set()
+        # Organize items into sections
+        added_item_names = set()  # To avoid duplicates
         for item in all_items:
-            section = item.get("Section__c", "Others")
+            section = item['Section__c']
             if section not in ordered_menu:
                 ordered_menu[section] = []
 
             if item['Name'] in added_item_names:
                 continue
 
-            if selected_category == "Veg" and item.get("Veg_NonVeg__c") not in ["Veg", "both"]:
+            # Apply category filter
+            veg_nonveg = item.get("Veg_NonVeg__c", "both")
+            if selected_category == "Veg" and veg_nonveg not in ["Veg", "both"]:
                 continue
-            if selected_category == "Non veg" and item.get("Veg_NonVeg__c") not in ["Non veg", "both"]:
+            if selected_category == "Non veg" and veg_nonveg not in ["Non veg", "both"]:
                 continue
 
             ordered_menu[section].append(item)
             added_item_names.add(item['Name'])
 
+        # Remove empty sections
         ordered_menu = {section: items for section, items in ordered_menu.items() if items}
         categories = ["All", "Veg", "Non veg"]
 
     except Exception as e:
         print(f"Error fetching menu data: {str(e)}")
-        # Fallback data with video and nutrition support
+        # Fallback data
         ordered_menu = {section: [] for section in SECTION_ORDER}
-        best_sellers = ["Chicken Biryani", "Paneer Butter Masala", "Veg Manchurian", "Prawn Fry"]
-        ordered_menu["Best Sellers"] = [{
-            "Name": name,
-            "Price__c": "12.99",
-            "Description__c": f"Popular {name}",
-            "Image1__c": "/static/placeholder.jpg",
-            "Video1__c": get_valid_video_path(name),
-            "Total_Ordered__c": 100,
-            "Veg_NonVeg__c": "Veg" if "Paneer" in name or "Veg" in name else "Non veg",
-            "IngredientsInfo__c": "Paneer, cream, tomatoes, spices" if "Paneer" in name else "Chicken, rice, spices, yogurt",
-            "Nutritional_Info__c": "Calories: 500-700, Protein: 20g" if "Paneer" in name else "Calories: 600-800, Protein: 30g",
-            "Allergens__c": "Dairy" if "Paneer" in name else "None"
-        } for name in best_sellers]
-        
+        best_sellers = [
+            {"Name": "Chicken Biryani", "Price__c": 12.99, "Description__c": "Popular Chicken Biryani", "Image1__c": "/static/placeholder.jpg", "Video1__c": get_valid_video_path("Chicken Biryani"), "Total_Ordered__c": 100, "Veg_NonVeg__c": "Non veg", "Section__c": "Best Sellers"},
+            {"Name": "Paneer Butter Masala", "Price__c": 11.99, "Description__c": "Popular Paneer Butter Masala", "Image1__c": "/static/placeholder.jpg", "Video1__c": get_valid_video_path("Paneer Butter Masala"), "Total_Ordered__c": 90, "Veg_NonVeg__c": "Veg", "Section__c": "Best Sellers"},
+            {"Name": "Veg Manchurian", "Price__c": 9.99, "Description__c": "Popular Veg Manchurian", "Image1__c": "/static/placeholder.jpg", "Video1__c": get_valid_video_path("Veg Manchurian"), "Total_Ordered__c": 80, "Veg_NonVeg__c": "Veg", "Section__c": "Best Sellers"},
+            {"Name": "Prawn Fry", "Price__c": 14.99, "Description__c": "Popular Prawn Fry", "Image1__c": "/static/placeholder.jpg", "Video1__c": get_valid_video_path("Prawn Fry"), "Total_Ordered__c": 70, "Veg_NonVeg__c": "Non veg", "Section__c": "Best Sellers"}
+        ]
+        ordered_menu["Best Sellers"] = best_sellers
         categories = ["All", "Veg", "Non veg"]
         referral_code = 'N/A'
         reward_points = 0
@@ -184,8 +175,8 @@ def menu():
 
 @menu_blueprint.route('/api/addons', methods=['GET'])
 def get_addons():
-    item_name = request.args.get('item_name')  
-    item_section = request.args.get('item_section')  
+    item_name = request.args.get('item_name')
+    item_section = request.args.get('item_section')
 
     if not item_name or not item_section:
         return jsonify({"success": False, "error": "Item name and section are required."}), 400
@@ -196,7 +187,7 @@ def get_addons():
             FROM Customization_Options__c
             WHERE Section__c = '{item_section}'
         """
-        result = sf.query(query)
+        result = sf.query_all(query)
         addons = result.get('records', [])
 
         if not addons:
@@ -205,14 +196,10 @@ def get_addons():
         formatted_addons = []
         for addon in addons:
             options = addon.get("Options__c", "")
-            if options:
-                options = options.split(", ")
-            else:
-                options = []
-
+            options = options.split(", ") if options else []
             formatted_addons.append({
-                "name": addon["Name"],
-                "type": addon["Customization_Type__c"],
+                "name": addon.get("Name", ""),
+                "type": addon.get("Customization_Type__c", ""),
                 "options": options,
                 "max_selections": addon.get("Max_Selections__c", 1),
                 "extra_charge": addon.get("Extra_Charge__c", False),
@@ -230,41 +217,41 @@ def add_to_cart():
     try:
         data = request.json
         item_name = data.get('itemName', '').strip()
-        item_price = data.get('itemPrice')
-        item_image = data.get('itemImage')
+        item_price = float(data.get('itemPrice', 0))
+        item_image = data.get('itemImage', '')
         addons = data.get('addons', [])
         instructions = data.get('instructions', '')
-        category = data.get('category')
-        section = data.get('section')
-        quantity = data.get('quantity', 1)
+        category = data.get('category', '')
+        section = data.get('section', '')
+        quantity = int(data.get('quantity', 1))
         customer_email = session.get('user_email')
 
-        if not item_name or not item_price:
-            return jsonify({"success": False, "error": "Item name and price are required."}), 400
-
-        if not customer_email:
-            return jsonify({"success": False, "error": "User email is required."}), 400
+        if not item_name or not item_price or not customer_email:
+            return jsonify({"success": False, "error": "Item name, price, and user email are required."}), 400
 
+        # Check if item already exists in cart
         query = f"""
-            SELECT Id, Quantity__c, Add_Ons__c, Add_Ons_Price__c, Instructions__c 
+            SELECT Id, Quantity__c, Add_Ons__c, Add_Ons_Price__c, Instructions__c, Price__c
             FROM Cart_Item__c
             WHERE Customer_Email__c = '{customer_email}' AND Name = '{item_name}'
         """
         result = sf.query(query)
         cart_items = result.get("records", [])
 
-        addons_price = sum(addon['price'] for addon in addons)
-        new_addons = "; ".join([f"{addon['name']} (${addon['price']})" for addon in addons])
+        addons_price = sum(float(addon.get('price', 0)) for addon in addons)
+        new_addons = "; ".join([f"{addon['name']} (${addon['price']})" for addon in addons]) if addons else "None"
 
         if cart_items:
-            cart_item_id = cart_items[0]['Id']
-            existing_quantity = cart_items[0]['Quantity__c']
-            existing_addons = cart_items[0].get('Add_Ons__c', "None")
-            existing_addons_price = cart_items[0].get('Add_Ons_Price__c', 0)
-            existing_instructions = cart_items[0].get('Instructions__c', "")
+            # Update existing cart item
+            cart_item = cart_items[0]
+            cart_item_id = cart_item['Id']
+            existing_quantity = int(cart_item.get('Quantity__c', 0))
+            existing_addons = cart_item.get('Add_Ons__c', "None")
+            existing_addons_price = float(cart_item.get('Add_Ons_Price__c', 0))
+            existing_instructions = cart_item.get('Instructions__c', "")
 
             combined_addons = existing_addons if existing_addons != "None" else ""
-            if new_addons:
+            if new_addons != "None":
                 combined_addons = f"{combined_addons}; {new_addons}".strip("; ")
 
             combined_instructions = existing_instructions
@@ -276,29 +263,27 @@ def add_to_cart():
                 float(addon.split("($")[1][:-1]) for addon in combined_addons_list if "($" in addon
             )
 
+            total_price = (existing_quantity + quantity) * item_price + combined_addons_price
+
             sf.Cart_Item__c.update(cart_item_id, {
                 "Quantity__c": existing_quantity + quantity,
                 "Add_Ons__c": combined_addons,
                 "Add_Ons_Price__c": combined_addons_price,
                 "Instructions__c": combined_instructions,
-                "Price__c": (existing_quantity + quantity) * item_price + combined_addons_price,
+                "Price__c": total_price,
                 "Category__c": category,
                 "Section__c": section
             })
         else:
-            addons_string = "None"
-            if addons:
-                addons_string = new_addons
-
+            # Create new cart item
             total_price = item_price * quantity + addons_price
-
             sf.Cart_Item__c.create({
                 "Name": item_name,
                 "Price__c": total_price,
                 "Base_Price__c": item_price,
                 "Quantity__c": quantity,
                 "Add_Ons_Price__c": addons_price,
-                "Add_Ons__c": addons_string,
+                "Add_Ons__c": new_addons,
                 "Image1__c": item_image,
                 "Customer_Email__c": customer_email,
                 "Instructions__c": instructions,
@@ -308,9 +293,8 @@ def add_to_cart():
 
         return jsonify({"success": True, "message": "Item added to cart successfully."})
 
-    except KeyError as e:
-        return jsonify({"success": False, "error": f"Missing required field: {str(e)}"}), 400
-
+    except ValueError as e:
+        return jsonify({"success": False, "error": f"Invalid data format: {str(e)}"}), 400
     except Exception as e:
         print(f"Error adding item to cart: {str(e)}")
-        return jsonify({"success": False, "error": "An error occurred while adding the item to the cart."}), 500
+        return jsonify({"success": False, "error": "An error occurred while adding the item to the cart."}), 500