fix: handle Get Items From buttons when is_return is set on document load

Author: Navin-S-R

erpnext/accounts/doctype/sales_invoice/sales_invoice.js

@@ -255,15 +255,16 @@ erpnext.accounts.SalesInvoiceController = class SalesInvoiceController extends (
 	}
 
 	toggle_get_items() {
-		// Show buttons only when pos view is inactive and not is_return
-		if (
-			cint(this.frm.doc.docstatus == 0) &&
-			this.frm.page.current_view_name !== "pos" &&
-			!this.frm.doc.is_return
-		) {
-			this.frm.cscript.sales_order_btn();
+		if (cint(this.frm.doc.docstatus == 0) && this.frm.page.current_view_name !== "pos") {
+			if (!this.frm.doc.is_return) {
+				this.frm.cscript.sales_order_btn();
+				this.frm.cscript.quotation_btn();
+			} else {
+				["Sales Order", "Quotation", "Timesheet"].forEach((label) => {
+					this.frm.remove_custom_button(label, "Get Items From");
+				});
+			}
 			this.frm.cscript.delivery_note_btn();
-			this.frm.cscript.quotation_btn();
 		} else {
 			["Sales Order", "Quotation", "Timesheet"].forEach((label) => {
 				this.frm.remove_custom_button(label, "Get Items From");