Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 1 | # Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors |
| 2 | # License: GNU General Public License v3. See license.txt |
| 3 | |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 4 | |
| 5 | import copy |
| 6 | import json |
| 7 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 8 | import frappe |
| 9 | from frappe import _ |
| 10 | from frappe.utils import cstr, flt |
Achilles Rasquinha | 56b2e12 | 2018-02-13 14:42:40 +0530 | [diff] [blame] | 11 | |
Sabu Siyad | f900a78 | 2023-10-17 17:05:44 +0530 | [diff] [blame] | 12 | from erpnext.utilities.product import get_item_codes_by_attributes |
| 13 | |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 14 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 15 | class ItemVariantExistsError(frappe.ValidationError): |
| 16 | pass |
| 17 | |
| 18 | |
| 19 | class InvalidItemAttributeValueError(frappe.ValidationError): |
| 20 | pass |
| 21 | |
| 22 | |
| 23 | class ItemTemplateCannotHaveStock(frappe.ValidationError): |
| 24 | pass |
| 25 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 26 | |
| 27 | @frappe.whitelist() |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 28 | def get_variant(template, args=None, variant=None, manufacturer=None, manufacturer_part_no=None): |
Sabu Siyad | f900a78 | 2023-10-17 17:05:44 +0530 | [diff] [blame] | 29 | """ |
| 30 | Validates Attributes and their Values, then looks for an exactly |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 31 | matching Item Variant |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 32 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 33 | :param item: Template Item |
| 34 | :param args: A dictionary with "Attribute" as key and "Attribute Value" as value |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 35 | """ |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 36 | item_template = frappe.get_doc("Item", template) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 37 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 38 | if item_template.variant_based_on == "Manufacturer" and manufacturer: |
| 39 | return make_variant_based_on_manufacturer(item_template, manufacturer, manufacturer_part_no) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 40 | |
Sabu Siyad | f900a78 | 2023-10-17 17:05:44 +0530 | [diff] [blame] | 41 | if isinstance(args, str): |
| 42 | args = json.loads(args) |
| 43 | |
| 44 | if not args: |
| 45 | frappe.throw(_("Please specify at least one attribute in the Attributes table")) |
| 46 | |
| 47 | return find_variant(template, args, variant) |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 48 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 49 | |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 50 | def make_variant_based_on_manufacturer(template, manufacturer, manufacturer_part_no): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 51 | """Make and return a new variant based on manufacturer and |
| 52 | manufacturer part no""" |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 53 | from frappe.model.naming import append_number_if_name_exists |
| 54 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 55 | variant = frappe.new_doc("Item") |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 56 | |
| 57 | copy_attributes_to_variant(template, variant) |
| 58 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 59 | variant.item_code = append_number_if_name_exists("Item", template.name) |
mergify[bot] | 4aa960b | 2023-12-19 14:59:33 +0530 | [diff] [blame] | 60 | variant.flags.ignore_mandatory = True |
| 61 | variant.save() |
| 62 | |
| 63 | if not frappe.db.exists( |
| 64 | "Item Manufacturer", {"item_code": variant.name, "manufacturer": manufacturer} |
| 65 | ): |
| 66 | manufacturer_doc = frappe.new_doc("Item Manufacturer") |
| 67 | manufacturer_doc.update( |
| 68 | { |
| 69 | "item_code": variant.name, |
| 70 | "manufacturer": manufacturer, |
| 71 | "manufacturer_part_no": manufacturer_part_no, |
| 72 | } |
| 73 | ) |
| 74 | |
| 75 | manufacturer_doc.flags.ignore_mandatory = True |
| 76 | manufacturer_doc.save(ignore_permissions=True) |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 77 | |
| 78 | return variant |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 79 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 80 | |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 81 | def validate_item_variant_attributes(item, args=None): |
Ankush Menat | 8fe5feb | 2021-11-04 19:48:32 +0530 | [diff] [blame] | 82 | if isinstance(item, str): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 83 | item = frappe.get_doc("Item", item) |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 84 | |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 85 | if not args: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 86 | args = {d.attribute.lower(): d.attribute_value for d in item.attributes} |
Rushabh Mehta | aed79e9 | 2016-06-02 17:49:16 +0530 | [diff] [blame] | 87 | |
Nabin Hait | aaf378e | 2017-12-13 18:40:52 +0530 | [diff] [blame] | 88 | attribute_values, numeric_values = get_attribute_values(item) |
Rushabh Mehta | aed79e9 | 2016-06-02 17:49:16 +0530 | [diff] [blame] | 89 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 90 | for attribute, value in args.items(): |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 91 | if not value: |
| 92 | continue |
| 93 | |
| 94 | if attribute.lower() in numeric_values: |
| 95 | numeric_attribute = numeric_values[attribute.lower()] |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 96 | validate_is_incremental(numeric_attribute, attribute, value, item.name) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 97 | |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 98 | else: |
| 99 | attributes_list = attribute_values.get(attribute.lower(), []) |
marination | 0df7f0f | 2020-06-01 11:56:33 +0530 | [diff] [blame] | 100 | validate_item_attribute_value(attributes_list, attribute, value, item.name, from_variant=True) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 101 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 102 | |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 103 | def validate_is_incremental(numeric_attribute, attribute, value, item): |
| 104 | from_range = numeric_attribute.from_range |
| 105 | to_range = numeric_attribute.to_range |
| 106 | increment = numeric_attribute.increment |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 107 | |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 108 | if increment == 0: |
| 109 | # defensive validation to prevent ZeroDivisionError |
| 110 | frappe.throw(_("Increment for Attribute {0} cannot be 0").format(attribute)) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 111 | |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 112 | is_in_range = from_range <= flt(value) <= to_range |
| 113 | precision = max(len(cstr(v).split(".")[-1].rstrip("0")) for v in (value, increment)) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 114 | # avoid precision error by rounding the remainder |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 115 | remainder = flt((flt(value) - from_range) % increment, precision) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 116 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 117 | is_incremental = remainder == 0 or remainder == increment |
Rushabh Mehta | aed79e9 | 2016-06-02 17:49:16 +0530 | [diff] [blame] | 118 | |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 119 | if not (is_in_range and is_incremental): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 120 | frappe.throw( |
| 121 | _( |
| 122 | "Value for Attribute {0} must be within the range of {1} to {2} in the increments of {3} for Item {4}" |
| 123 | ).format(attribute, from_range, to_range, increment, item), |
| 124 | InvalidItemAttributeValueError, |
| 125 | title=_("Invalid Attribute"), |
| 126 | ) |
Rohit Waghchaure | 6500ef4 | 2016-12-15 18:24:32 +0530 | [diff] [blame] | 127 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 128 | |
| 129 | def validate_item_attribute_value( |
| 130 | attributes_list, attribute, attribute_value, item, from_variant=True |
| 131 | ): |
| 132 | allow_rename_attribute_value = frappe.db.get_single_value( |
| 133 | "Item Variant Settings", "allow_rename_attribute_value" |
| 134 | ) |
Vishal Dhayagude | 668ec25 | 2018-03-07 15:31:08 +0530 | [diff] [blame] | 135 | if allow_rename_attribute_value: |
| 136 | pass |
| 137 | elif attribute_value not in attributes_list: |
marination | 0df7f0f | 2020-06-01 11:56:33 +0530 | [diff] [blame] | 138 | if from_variant: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 139 | frappe.throw( |
| 140 | _("{0} is not a valid Value for Attribute {1} of Item {2}.").format( |
| 141 | frappe.bold(attribute_value), frappe.bold(attribute), frappe.bold(item) |
| 142 | ), |
| 143 | InvalidItemAttributeValueError, |
| 144 | title=_("Invalid Value"), |
| 145 | ) |
marination | 0df7f0f | 2020-06-01 11:56:33 +0530 | [diff] [blame] | 146 | else: |
marination | 91dfd00 | 2020-06-17 19:05:40 +0530 | [diff] [blame] | 147 | msg = _("The value {0} is already assigned to an existing Item {1}.").format( |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 148 | frappe.bold(attribute_value), frappe.bold(item) |
| 149 | ) |
| 150 | msg += "<br>" + _( |
| 151 | "To still proceed with editing this Attribute Value, enable {0} in Item Variant Settings." |
| 152 | ).format(frappe.bold("Allow Rename Attribute Value")) |
marination | 0df7f0f | 2020-06-01 11:56:33 +0530 | [diff] [blame] | 153 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 154 | frappe.throw(msg, InvalidItemAttributeValueError, title=_("Edit Not Allowed")) |
| 155 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 156 | |
Nabin Hait | aaf378e | 2017-12-13 18:40:52 +0530 | [diff] [blame] | 157 | def get_attribute_values(item): |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 158 | if not frappe.flags.attribute_values: |
| 159 | attribute_values = {} |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 160 | numeric_values = {} |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 161 | for t in frappe.get_all("Item Attribute Value", fields=["parent", "attribute_value"]): |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 162 | attribute_values.setdefault(t.parent.lower(), []).append(t.attribute_value) |
| 163 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 164 | for t in frappe.get_all( |
| 165 | "Item Variant Attribute", |
Nabin Hait | aaf378e | 2017-12-13 18:40:52 +0530 | [diff] [blame] | 166 | fields=["attribute", "from_range", "to_range", "increment"], |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 167 | filters={"numeric_values": 1, "parent": item.variant_of}, |
| 168 | ): |
Nabin Hait | aaf378e | 2017-12-13 18:40:52 +0530 | [diff] [blame] | 169 | numeric_values[t.attribute.lower()] = t |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 170 | |
| 171 | frappe.flags.attribute_values = attribute_values |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 172 | frappe.flags.numeric_values = numeric_values |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 173 | |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 174 | return frappe.flags.attribute_values, frappe.flags.numeric_values |
Rushabh Mehta | b8bdfbc | 2016-07-15 12:40:47 +0530 | [diff] [blame] | 175 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 176 | |
Nabin Hait | 6b068e1 | 2015-12-31 13:20:32 +0530 | [diff] [blame] | 177 | def find_variant(template, args, variant_item_code=None): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 178 | possible_variants = [ |
| 179 | i for i in get_item_codes_by_attributes(args, template) if i != variant_item_code |
| 180 | ] |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 181 | |
| 182 | for variant in possible_variants: |
| 183 | variant = frappe.get_doc("Item", variant) |
| 184 | |
| 185 | if len(args.keys()) == len(variant.get("attributes")): |
| 186 | # has the same number of attributes and values |
| 187 | # assuming no duplication as per the validation in Item |
| 188 | match_count = 0 |
| 189 | |
| 190 | for attribute, value in args.items(): |
| 191 | for row in variant.attributes: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 192 | if row.attribute == attribute and row.attribute_value == cstr(value): |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 193 | # this row matches |
| 194 | match_count += 1 |
| 195 | break |
| 196 | |
| 197 | if match_count == len(args.keys()): |
| 198 | return variant.name |
| 199 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 200 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 201 | @frappe.whitelist() |
| 202 | def create_variant(item, args): |
Ankush Menat | 8fe5feb | 2021-11-04 19:48:32 +0530 | [diff] [blame] | 203 | if isinstance(args, str): |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 204 | args = json.loads(args) |
| 205 | |
| 206 | template = frappe.get_doc("Item", item) |
| 207 | variant = frappe.new_doc("Item") |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 208 | variant.variant_based_on = "Item Attribute" |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 209 | variant_attributes = [] |
| 210 | |
| 211 | for d in template.attributes: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 212 | variant_attributes.append({"attribute": d.attribute, "attribute_value": args.get(d.attribute)}) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 213 | |
| 214 | variant.set("attributes", variant_attributes) |
| 215 | copy_attributes_to_variant(template, variant) |
Prateeksha Singh | 89cec18 | 2017-05-19 12:35:36 +0530 | [diff] [blame] | 216 | make_variant_item_code(template.item_code, template.item_name, variant) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 217 | |
| 218 | return variant |
| 219 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 220 | |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 221 | @frappe.whitelist() |
| 222 | def enqueue_multiple_variant_creation(item, args): |
| 223 | # There can be innumerable attribute combinations, enqueue |
Ankush Menat | 8fe5feb | 2021-11-04 19:48:32 +0530 | [diff] [blame] | 224 | if isinstance(args, str): |
Ameya Shenoy | 935f4a4 | 2018-07-03 10:48:59 +0530 | [diff] [blame] | 225 | variants = json.loads(args) |
| 226 | total_variants = 1 |
| 227 | for key in variants: |
| 228 | total_variants *= len(variants[key]) |
| 229 | if total_variants >= 600: |
Anurag Mishra | 841d852 | 2019-07-03 15:15:08 +0530 | [diff] [blame] | 230 | frappe.throw(_("Please do not create more than 500 items at a time")) |
Ameya Shenoy | 935f4a4 | 2018-07-03 10:48:59 +0530 | [diff] [blame] | 231 | return |
Rushabh Mehta | 8705371 | 2018-08-16 09:22:33 +0530 | [diff] [blame] | 232 | if total_variants < 10: |
| 233 | return create_multiple_variants(item, args) |
| 234 | else: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 235 | frappe.enqueue( |
| 236 | "erpnext.controllers.item_variant.create_multiple_variants", |
| 237 | item=item, |
| 238 | args=args, |
| 239 | now=frappe.flags.in_test, |
| 240 | ) |
| 241 | return "queued" |
| 242 | |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 243 | |
| 244 | def create_multiple_variants(item, args): |
Rushabh Mehta | 8705371 | 2018-08-16 09:22:33 +0530 | [diff] [blame] | 245 | count = 0 |
Ankush Menat | 8fe5feb | 2021-11-04 19:48:32 +0530 | [diff] [blame] | 246 | if isinstance(args, str): |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 247 | args = json.loads(args) |
| 248 | |
| 249 | args_set = generate_keyed_value_combinations(args) |
| 250 | |
| 251 | for attribute_values in args_set: |
| 252 | if not get_variant(item, args=attribute_values): |
| 253 | variant = create_variant(item, attribute_values) |
| 254 | variant.save() |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 255 | count += 1 |
Rushabh Mehta | 8705371 | 2018-08-16 09:22:33 +0530 | [diff] [blame] | 256 | |
| 257 | return count |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 258 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 259 | |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 260 | def generate_keyed_value_combinations(args): |
| 261 | """ |
| 262 | From this: |
| 263 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 264 | args = {"attr1": ["a", "b", "c"], "attr2": ["1", "2"], "attr3": ["A"]} |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 265 | |
| 266 | To this: |
| 267 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 268 | [ |
| 269 | {u'attr1': u'a', u'attr2': u'1', u'attr3': u'A'}, |
| 270 | {u'attr1': u'b', u'attr2': u'1', u'attr3': u'A'}, |
| 271 | {u'attr1': u'c', u'attr2': u'1', u'attr3': u'A'}, |
| 272 | {u'attr1': u'a', u'attr2': u'2', u'attr3': u'A'}, |
| 273 | {u'attr1': u'b', u'attr2': u'2', u'attr3': u'A'}, |
| 274 | {u'attr1': u'c', u'attr2': u'2', u'attr3': u'A'} |
| 275 | ] |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 276 | |
| 277 | """ |
| 278 | # Return empty list if empty |
| 279 | if not args: |
| 280 | return [] |
| 281 | |
| 282 | # Turn `args` into a list of lists of key-value tuples: |
| 283 | # [ |
| 284 | # [(u'attr2', u'1'), (u'attr2', u'2')], |
| 285 | # [(u'attr3', u'A')], |
| 286 | # [(u'attr1', u'a'), (u'attr1', u'b'), (u'attr1', u'c')] |
| 287 | # ] |
| 288 | key_value_lists = [[(key, val) for val in args[key]] for key in args.keys()] |
| 289 | |
| 290 | # Store the first, but as objects |
| 291 | # [{u'attr2': u'1'}, {u'attr2': u'2'}] |
| 292 | results = key_value_lists.pop(0) |
| 293 | results = [{d[0]: d[1]} for d in results] |
| 294 | |
| 295 | # Iterate the remaining |
| 296 | # Take the next list to fuse with existing results |
| 297 | for l in key_value_lists: |
| 298 | new_results = [] |
| 299 | for res in results: |
| 300 | for key_val in l: |
| 301 | # create a new clone of object in result |
| 302 | obj = copy.deepcopy(res) |
| 303 | # to be used with every incoming new value |
| 304 | obj[key_val[0]] = key_val[1] |
| 305 | # and pushed into new_results |
| 306 | new_results.append(obj) |
| 307 | results = new_results |
| 308 | |
| 309 | return results |
| 310 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 311 | |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 312 | def copy_attributes_to_variant(item, variant): |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 313 | # copy non no-copy fields |
| 314 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 315 | exclude_fields = [ |
| 316 | "naming_series", |
| 317 | "item_code", |
| 318 | "item_name", |
| 319 | "published_in_website", |
| 320 | "opening_stock", |
| 321 | "variant_of", |
| 322 | "valuation_rate", |
| 323 | ] |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 324 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 325 | if item.variant_based_on == "Manufacturer": |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 326 | # don't copy manufacturer values if based on part no |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 327 | exclude_fields += ["manufacturer", "manufacturer_part_no"] |
Rushabh Mehta | a07c43f | 2017-03-21 17:48:34 +0100 | [diff] [blame] | 328 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 329 | allow_fields = [d.field_name for d in frappe.get_all("Variant Field", fields=["field_name"])] |
Nabin Hait | 945f502 | 2017-09-29 15:11:50 +0530 | [diff] [blame] | 330 | if "variant_based_on" not in allow_fields: |
| 331 | allow_fields.append("variant_based_on") |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 332 | for field in item.meta.fields: |
tundebabzy | 6015f0f | 2017-07-04 11:13:02 +0100 | [diff] [blame] | 333 | # "Table" is part of `no_value_field` but we shouldn't ignore tables |
Rohit Waghchaure | 0e28fcc | 2017-08-29 18:15:57 +0530 | [diff] [blame] | 334 | if (field.reqd or field.fieldname in allow_fields) and field.fieldname not in exclude_fields: |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 335 | if variant.get(field.fieldname) != item.get(field.fieldname): |
Nabin Hait | 945f502 | 2017-09-29 15:11:50 +0530 | [diff] [blame] | 336 | if field.fieldtype == "Table": |
| 337 | variant.set(field.fieldname, []) |
| 338 | for d in item.get(field.fieldname): |
| 339 | row = copy.deepcopy(d) |
| 340 | if row.get("name"): |
| 341 | row.name = None |
| 342 | variant.append(field.fieldname, row) |
| 343 | else: |
| 344 | variant.set(field.fieldname, item.get(field.fieldname)) |
Nabin Hait | c314485 | 2017-09-28 18:55:40 +0530 | [diff] [blame] | 345 | |
Rohit Waghchaure | 0eef3f6 | 2018-06-13 13:06:25 +0530 | [diff] [blame] | 346 | variant.variant_of = item.name |
Anurag Mishra | 62d58ac | 2019-05-30 14:04:08 +0530 | [diff] [blame] | 347 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 348 | if "description" not in allow_fields: |
hiousi | 38de994 | 2018-04-24 08:40:45 +0200 | [diff] [blame] | 349 | if not variant.description: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 350 | variant.description = "" |
Rucha Mahabal | c6b548b | 2019-09-24 19:17:13 +0530 | [diff] [blame] | 351 | else: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 352 | if item.variant_based_on == "Item Attribute": |
hiousi | 38de994 | 2018-04-24 08:40:45 +0200 | [diff] [blame] | 353 | if variant.attributes: |
Anurag Mishra | 62d58ac | 2019-05-30 14:04:08 +0530 | [diff] [blame] | 354 | attributes_description = item.description + " " |
hiousi | 38de994 | 2018-04-24 08:40:45 +0200 | [diff] [blame] | 355 | for d in variant.attributes: |
| 356 | attributes_description += "<div>" + d.attribute + ": " + cstr(d.attribute_value) + "</div>" |
Prateeksha Singh | 8f43d25 | 2017-11-16 18:06:26 +0530 | [diff] [blame] | 357 | |
hiousi | 38de994 | 2018-04-24 08:40:45 +0200 | [diff] [blame] | 358 | if attributes_description not in variant.description: |
Rucha Mahabal | c6b548b | 2019-09-24 19:17:13 +0530 | [diff] [blame] | 359 | variant.description = attributes_description |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 360 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 361 | |
Prateeksha Singh | 89cec18 | 2017-05-19 12:35:36 +0530 | [diff] [blame] | 362 | def make_variant_item_code(template_item_code, template_item_name, variant): |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 363 | """Uses template's item code and abbreviations to make variant's item code""" |
| 364 | if variant.item_code: |
| 365 | return |
| 366 | |
| 367 | abbreviations = [] |
| 368 | for attr in variant.attributes: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 369 | item_attribute = frappe.db.sql( |
| 370 | """select i.numeric_values, v.abbr |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 371 | from `tabItem Attribute` i left join `tabItem Attribute Value` v |
| 372 | on (i.name=v.parent) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 373 | where i.name=%(attribute)s and (v.attribute_value=%(attribute_value)s or i.numeric_values = 1)""", |
| 374 | {"attribute": attr.attribute, "attribute_value": attr.attribute_value}, |
| 375 | as_dict=True, |
| 376 | ) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 377 | |
| 378 | if not item_attribute: |
Faris Ansari | ab14892 | 2019-05-03 13:57:20 +0530 | [diff] [blame] | 379 | continue |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 380 | # frappe.throw(_('Invalid attribute {0} {1}').format(frappe.bold(attr.attribute), |
| 381 | # frappe.bold(attr.attribute_value)), title=_('Invalid Attribute'), |
| 382 | # exc=InvalidItemAttributeValueError) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 383 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 384 | abbr_or_value = ( |
| 385 | cstr(attr.attribute_value) if item_attribute[0].numeric_values else item_attribute[0].abbr |
| 386 | ) |
Rohit Waghchaure | 0169341 | 2017-03-09 17:02:55 +0530 | [diff] [blame] | 387 | abbreviations.append(abbr_or_value) |
Anand Doshi | 099bbbd | 2015-09-02 11:18:32 +0530 | [diff] [blame] | 388 | |
| 389 | if abbreviations: |
Rushabh Mehta | 95383bb | 2016-07-15 15:11:46 +0530 | [diff] [blame] | 390 | variant.item_code = "{0}-{1}".format(template_item_code, "-".join(abbreviations)) |
Prateeksha Singh | 89cec18 | 2017-05-19 12:35:36 +0530 | [diff] [blame] | 391 | variant.item_name = "{0}-{1}".format(template_item_name, "-".join(abbreviations)) |
Rushabh Mehta | d5c6416 | 2017-11-14 15:27:28 +0530 | [diff] [blame] | 392 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 393 | |
Rushabh Mehta | d5c6416 | 2017-11-14 15:27:28 +0530 | [diff] [blame] | 394 | @frappe.whitelist() |
| 395 | def create_variant_doc_for_quick_entry(template, args): |
| 396 | variant_based_on = frappe.db.get_value("Item", template, "variant_based_on") |
| 397 | args = json.loads(args) |
| 398 | if variant_based_on == "Manufacturer": |
| 399 | variant = get_variant(template, **args) |
| 400 | else: |
| 401 | existing_variant = get_variant(template, args) |
| 402 | if existing_variant: |
| 403 | return existing_variant |
| 404 | else: |
| 405 | variant = create_variant(template, args=args) |
| 406 | variant.name = variant.item_code |
| 407 | validate_item_variant_attributes(variant, args) |
| 408 | return variant.as_dict() |