Get specifics field in get_query function

narcisonunez · March 11, 2015, 8:16pm

Hello Everyone,

I’m trying to get 2 specific fields from my doctype when a use the link field get_query function.

cur_frm.fields_dict[‘distrito_no’].get_query = function(doc, cdt, cdn) {
return {
query: “app.app.doctype.doctype_name.doctype_name.get_distrito”,
filters:{ ‘codigo_regional’: doc.regional_no }
}
}

I have a server side function

def get_distrito(doctype, txt, searchfield, start, page_len, filters):
return frappe.db.sql(“”“SELECT codigo_distrito, nombre_distrito
FROM tabDistrito
WHERE codigo_regional = {0} “””.format(filters.get(“codigo_regional”)))

the result is exactly what i want but when i select the value, the field is cleaned by itself.

What am i doing wrong?

Help!!

rmehta · March 13, 2015, 7:17am

also set as_dict=1 in your frappe.db.sql method.

Do you have an error trace in your js console?

codeag · November 2, 2016, 1:18pm

I just figured this out @geekroot. This is what happened to me and I have a similar issue.

You need to select the name column of your custom DocType as the first column. When selecting the item in the dropdown it’s “cleaning itself” because a validation function is run to validate the selected item. It validates the item by it’s first selected column which is expected to be the name aka the primary key of your table.

So your query should look like:
select name, codigo_distrito, nombre_distrito from...

@rmehta:

github.com

frappe/frappe/blob/develop/frappe/public/js/frappe/form/script_manager.js#L138


      
          			}
          		});
          
          
		handlers.old_style.forEach((_function) => {
          			if (event_name === "setup") {
          				// setup must be called immediately
          				runner(_function, true);
          			} else {
          				tasks.push(() => runner(_function, true));
          			}
          		});
          
          
		// run them serially
          		return frappe.run_serially(tasks);
          	}
          	has_handlers(event_name, doctype) {
          		let handlers = this.get_handlers(event_name, doctype);
          		return handlers && (handlers.old_style.length || handlers.new_style.length);
          	}
          	get_handlers(event_name, doctype) {
          		// returns list of all functions to be called (old style and new style)

This is the relevant function. Is there a way to manipulate the value that is sent to this function to have other values but the name as the first column?

Or maybe there’s already an inbuilt solution for the validate_link function on server-side so it doesn’t assume that the value it is provided is the name of the doc but some other column?

github.com

frappe/frappe/blob/develop/frappe/desk/form/utils.py#L19


      
          import frappe.desk.form.meta
          from frappe import _
          from frappe.core.doctype.file.utils import extract_images_from_html
          from frappe.desk.form.document_follow import follow_document
          
          
if TYPE_CHECKING:
          	from frappe.core.doctype.comment.comment import Comment
          
          

          
@frappe.whitelist(methods=["DELETE", "POST"])
          def remove_attach():
          	"""remove attachment"""
          	fid = frappe.form_dict.get("fid")
          	frappe.delete_doc("File", fid)
          
          

          
@frappe.whitelist(methods=["POST", "PUT"])
          def add_comment(
          	reference_doctype: str, reference_name: str, content: str, comment_email: str, comment_by: str
          ) -> "Comment":
          	"""Allow logged user with permission to read document to add a comment"""