Update menu.py

menu.py

@@ -1,6 +1,6 @@
 from flask import Blueprint, render_template, request, session, jsonify, redirect, url_for
-import os
 from salesforce import get_salesforce_connection
+import os
 
 menu_blueprint = Blueprint('menu', __name__)
 
@@ -15,23 +15,31 @@ 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):
-    with open(PLACEHOLDER_PATH, 'wb') as f:
-        f.close()
+    open(PLACEHOLDER_PATH, 'wb').close()
     print(f"Created placeholder video at {PLACEHOLDER_PATH}")
 
 def get_valid_video_path(item_name, video_url=None):
-    """Get valid video path for item with placeholder fallback."""
+    """
+    Get valid video path for item with placeholder fallback
+    Priority: 1. Video1__c from Salesforce 2. placeholder.mp4
+    """
+    # First try: Video1__c from Salesforce if provided
     if video_url:
+        # If it's a complete URL (http/https)
         if video_url.startswith(('http://', 'https://')):
             return video_url
+        # If it's a relative path (/videos/xxx.mp4)
         elif video_url.startswith('/'):
             return video_url
+        # If it's a Salesforce File ID (starts with '069')
         elif video_url.startswith('069'):
-            return f"https://biryanihub-dev-ed.develop.my.salesforce.com/sfc/servlet.shepherd/version/download/{video_url}"
+            return f"https://yourdomain.my.salesforce.com/sfc/servlet.shepherd/version/download/{video_url}"
+    
+    # Final fallback: placeholder.mp4
     if not os.path.exists(PLACEHOLDER_PATH):
-        with open(PLACEHOLDER_PATH, 'wb') as f:
-            f.close()
+        open(PLACEHOLDER_PATH, 'wb').close()
         print(f"Created missing placeholder video at {PLACEHOLDER_PATH}")
+    
     return f"/static/{PLACEHOLDER_VIDEO}"
 
 @menu_blueprint.route("/menu", methods=["GET", "POST"])
@@ -39,11 +47,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 and user_name:
+
+        if user_email:
             session['user_email'] = user_email
             session['user_name'] = user_name
         else:
@@ -53,95 +61,122 @@ def menu():
 
     first_letter = user_name[0].upper() if user_name else "A"
 
-    # 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.get('records'):
-        return redirect(url_for('login'))
-
-    referral_code = user_result['records'][0].get('Referral__c', 'N/A')
-    reward_points = user_result['records'][0].get('Reward_Points__c', 0)
-
-    # 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.get('totalSize', 0)
-
-    # Fetch all Menu_Item__c records with only existing fields
-    menu_query = """
-        SELECT Name, Price__c, Description__c, Image1__c, Image2__c, 
-               Veg_NonVeg__c, Section__c, Total_Ordered__c, Video1__c
-        FROM Menu_Item__c
-    """
-    menu_result = sf.query_all(menu_query)
-    food_items = menu_result.get('records', [])
-
-    # Process menu items
-    for item in food_items:
-        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', "Others")
-        item['Description__c'] = item.get('Description__c', "No description available")
-        # Add default values for missing fields
-        item['IngredientsInfo__c'] = "Not specified"
-        item['NutritionalInfo__c'] = "Not available"
-        item['Allergens__c'] = "None listed"
-
-    # Fetch all Custom_Dish__c records with only existing fields
-    custom_dish_query = """
-        SELECT Name, Price__c, Description__c, Image1__c, Image2__c, 
-               Veg_NonVeg__c, Section__c, Total_Ordered__c
-        FROM Custom_Dish__c
-        WHERE CreatedDate >= LAST_N_DAYS:7
-    """
-    custom_dish_result = sf.query_all(custom_dish_query)
-    custom_dishes = custom_dish_result.get('records', [])
-
-    # Process custom dishes
-    for item in custom_dishes:
-        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', "Customized dish")
-        item['Description__c'] = item.get('Description__c', "No description available")
-        # Add default values for missing fields
-        item['IngredientsInfo__c'] = "Not specified"
-        item['NutritionalInfo__c'] = "Not available"
-        item['Allergens__c'] = "None listed"
-
-    # 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['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]
-
-    # Organize items into sections
-    added_item_names = set()
-    for item in all_items:
-        section = item['Section__c']
-        if section not in ordered_menu:
-            ordered_menu[section] = []
-
-        if item['Name'] in added_item_names:
-            continue
-
-        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 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"]
+    try:
+        # 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']:
+            return redirect(url_for('login'))
+
+        referral_code = user_result['records'][0].get('Referral__c', 'N/A')
+        reward_points = user_result['records'][0].get('Reward_Points__c', 0)
+
+        # 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']
+
+        # First try basic query with essential fields
+        base_fields = [
+            "Name", "Price__c", "Description__c", "Image1__c", "Image2__c",
+            "Veg_NonVeg__c", "Section__c", "Total_Ordered__c", "Video1__c"
+        ]
+        
+        # Try to include nutrition fields if they exist
+        try:
+            describe_result = sf.Menu_Item__c.describe()
+            available_fields = [field['name'] for field in describe_result['fields']]
+            
+            nutrition_fields = []
+            if 'IngredientsInfo__c' in available_fields:
+                nutrition_fields.append('IngredientsInfo__c')
+            if 'Nutritional_Info__c' in available_fields:
+                nutrition_fields.append('Nutritional_Info__c')
+            if 'Allergens__c' in available_fields:
+                nutrition_fields.append('Allergens__c')
+                
+            query_fields = base_fields + nutrition_fields
+        except Exception as e:
+            print(f"Error checking for nutrition fields: {str(e)}")
+            query_fields = base_fields
+
+        # Build and execute the query
+        menu_query = f"SELECT {', '.join(query_fields)} FROM Menu_Item__c"
+        result = sf.query(menu_query)
+        food_items = result['records'] if 'records' in result else []
+
+        # Process items with fallback values
+        for item in food_items:
+            # Ensure required fields have values
+            item['Total_Ordered__c'] = item.get('Total_Ordered__c', 0)
+            item['Video1__c'] = get_valid_video_path(item['Name'], item.get('Video1__c'))
+            
+            # Add fallback values for nutrition fields
+            item['IngredientsInfo__c'] = item.get('IngredientsInfo__c', 'Ingredients information not available')
+            item['Nutritional_Info__c'] = item.get('Nutritional_Info__c', 'Nutritional information not available')
+            item['Allergens__c'] = item.get('Allergens__c', 'Allergen information not available')
+
+        # Process best sellers
+        best_sellers = sorted(food_items, key=lambda x: x.get("Total_Ordered__c", 0), reverse=True)
+        
+        # Apply category filter
+        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"]]
+
+        best_sellers = best_sellers[:4]
+        ordered_menu = {section: [] for section in SECTION_ORDER}
+        
+        if best_sellers:
+            ordered_menu["Best Sellers"] = best_sellers
+
+        # Organize items by section
+        added_item_names = set()
+        for item in food_items:
+            section = item.get("Section__c", "Others")
+            if section not in ordered_menu:
+                ordered_menu[section] = []
+
+            if item['Name'] in added_item_names:
+                continue
+
+            # Apply category filter to all items
+            if selected_category == "Veg" and item.get("Veg_NonVeg__c") not in ["Veg", "both"]:
+                continue
+            if selected_category == "Non veg" and item.get("Veg_NonVeg__c") 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
+        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": "Ingredients information not available",
+            "Nutritional_Info__c": "Nutritional information not available",
+            "Allergens__c": "Allergen information not available"
+        } for name in best_sellers]
+        
+        categories = ["All", "Veg", "Non veg"]
+        referral_code = 'N/A'
+        reward_points = 0
+        cart_item_count = 0
 
     return render_template(
         "menu.html",
@@ -157,8 +192,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
@@ -169,7 +204,7 @@ def get_addons():
             FROM Customization_Options__c
             WHERE Section__c = '{item_section}'
         """
-        result = sf.query_all(query)
+        result = sf.query(query)
         addons = result.get('records', [])
 
         if not addons:
@@ -178,10 +213,14 @@ def get_addons():
         formatted_addons = []
         for addon in addons:
             options = addon.get("Options__c", "")
-            options = options.split(", ") if options else []
+            if options:
+                options = options.split(", ")
+            else:
+                options = []
+
             formatted_addons.append({
-                "name": addon.get("Name", ""),
-                "type": addon.get("Customization_Type__c", ""),
+                "name": addon["Name"],
+                "type": addon["Customization_Type__c"],
                 "options": options,
                 "max_selections": addon.get("Max_Selections__c", 1),
                 "extra_charge": addon.get("Extra_Charge__c", False),
@@ -199,39 +238,41 @@ def add_to_cart():
     try:
         data = request.json
         item_name = data.get('itemName', '').strip()
-        item_price = float(data.get('itemPrice', 0))
-        item_image = data.get('itemImage', '')
+        item_price = data.get('itemPrice')
+        item_image = data.get('itemImage')
         addons = data.get('addons', [])
         instructions = data.get('instructions', '')
-        category = data.get('category', '')
-        section = data.get('section', '')
-        quantity = int(data.get('quantity', 1))
+        category = data.get('category')
+        section = data.get('section')
+        quantity = data.get('quantity', 1)
         customer_email = session.get('user_email')
 
-        if not item_name or not item_price or not customer_emailKILL
-            return jsonify({"success": False, "error": "Item name, price, and user email are required."}), 400
+        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
 
         query = f"""
-            SELECT Id, Quantity__c, Add_Ons__c, Add_Ons_Price__c, Instructions__c, Price__c
+            SELECT Id, Quantity__c, Add_Ons__c, Add_Ons_Price__c, Instructions__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(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"
+        addons_price = sum(addon['price'] for addon in addons)
+        new_addons = "; ".join([f"{addon['name']} (${addon['price']})" for addon in addons])
 
         if cart_items:
-            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', "")
+            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', "")
 
             combined_addons = existing_addons if existing_addons != "None" else ""
-            if new_addons != "None":
+            if new_addons:
                 combined_addons = f"{combined_addons}; {new_addons}".strip("; ")
 
             combined_instructions = existing_instructions
@@ -243,26 +284,29 @@ 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": total_price,
+                "Price__c": (existing_quantity + quantity) * item_price + combined_addons_price,
                 "Category__c": category,
                 "Section__c": section
             })
         else:
+            addons_string = "None"
+            if addons:
+                addons_string = new_addons
+
             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": new_addons,
+                "Add_Ons__c": addons_string,
                 "Image1__c": item_image,
                 "Customer_Email__c": customer_email,
                 "Instructions__c": instructions,
@@ -270,15 +314,11 @@ def add_to_cart():
                 "Section__c": section
             })
 
-        # Fetch updated cart for UI update
-        cart_query = f"SELECT Name, Quantity__c FROM Cart_Item__c WHERE Customer_Email__c = '{customer_email}'"
-        cart_result = sf.query_all(cart_query)
-        cart = [{"itemName": item["Name"], "quantity": item["Quantity__c"]} for item in cart_result.get("records", [])]
-        
-        return jsonify({"success": True, "message": "Item added to cart successfully.", "cart": 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