Error with the create button

ragav · October 27, 2016, 7:39am

Hi Guys ,

I am planning to make a new doc from my custom app , vave with the following code which returns an error , please help

frappe.ui.form.on("VAVE", "refresh", function(frm){
     {
      frm.add_custom_button(__("Create dta"), function() {
        frappe.model.open_mapped_doc({
                        method : "vave.vave.doctype.dta.make_dta",
                        frm : cur_frm
                })
         })
    }
});

when I click on the create dta button , I am getting the below error displayed

Traceback (most recent call last):
  File "/home/ommi/frappe-bench/apps/frappe/frappe/app.py", line 55, in application
    response = frappe.handler.handle()
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 19, in handle
    execute_cmd(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 30, in execute_cmd
    method = get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 114, in get_attr
    method = frappe.get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/__init__.py", line 856, in get_attr
    return getattr(get_module(modulename), methodname)
AttributeError: 'module' object has no attribute 'make_dta'

Thanks in advance

rohit_w · October 27, 2016, 7:50am

Hi @ragav,

It seems that path is not correct, it should be like

method : "vave.vave.doctype.dta.dta.make_dta",

I hope dta is your doctype

Thanks

ragav · October 27, 2016, 8:07am

Hi @rohit_w , Yes you are correct dta is my doctype , I changed my code , but still the same error , could you guide me please ?

Thanks in advance

rohit_w · October 27, 2016, 8:11am

Hi @ragav,

Can you share me your github repo, so its easy to me to find the issue.

Thanks

ragav · October 27, 2016, 8:17am

So kind of you @rohit_w . This is the link to my repo GitHub - Niville/myerp1 at develop

I created a new doctype dta in my local inside the vave module which I am yet to push to my repo .

Thanks a ton for helping

rohit_w · October 27, 2016, 8:22am

Hi @ragav,

I have to check the file dta.py, It will be great if you push the module vave to your repo so I can easily checks the file dta.py.

Thanks

ragav · October 27, 2016, 8:46am

Many thanks for the continued support @rohit_w . Link to the repo containing both vave and dta .

Thanks a lot for the help

rohit_w · October 27, 2016, 9:07am

Hi @ragav,

Path is correct but method make_dta is not present in the dta.py file, please check below

github.com

Niville/vave/blob/master/vave/vave/doctype/dta/dta.py

# -*- coding: utf-8 -*-
# Copyright (c) 2015, value added value engineering and contributors
# For license information, please see license.txt

from __future__ import unicode_literals
import frappe
from frappe.model.document import Document

class dta(Document):
	pass


@frappe.whitelist()
def make_dta(source_name, target_doc=None):

doclist = get_mapped_doc("dta", source_name,		{
		"dta": {
			"doctype": "vave",
			
		},

This file has been truncated. show original

You need to write method in the dat.py file, for example check below js and py file

Js file

github.com

frappe/erpnext/blob/develop/erpnext/buying/doctype/supplier_quotation/supplier_quotation.js#L33-L39


      
          					erpnext.utils.map_current_doc({
          						method: "erpnext.stock.doctype.material_request.material_request.make_supplier_quotation",
          						source_doctype: "Material Request",
          						target: me.frm,
          						setters: {
          							schedule_date: undefined,
          							status: undefined

Py file

github.com

frappe/erpnext/blob/develop/erpnext/buying/doctype/supplier_quotation/supplier_quotation.py#L70-L104


      
          		named_place: DF.Data | None
          		naming_series: DF.Literal["PUR-SQTN-.YYYY.-"]
          		net_total: DF.Currency
          		opportunity: DF.Link | None
          		other_charges_calculation: DF.LongText | None
          		plc_conversion_rate: DF.Float
          		price_list_currency: DF.Link | None
          		pricing_rules: DF.Table[PricingRuleDetail]
          		project: DF.Link | None
          		quotation_number: DF.Data | None
          		rounded_total: DF.Currency
          		rounding_adjustment: DF.Currency
          		select_print_heading: DF.Link | None
          		shipping_address: DF.Link | None
          		shipping_address_display: DF.SmallText | None
          		shipping_rule: DF.Link | None
          		status: DF.Literal["", "Draft", "Submitted", "Stopped", "Cancelled", "Expired"]
          		supplier: DF.Link
          		supplier_address: DF.Link | None
          		supplier_name: DF.Data | None

This file has been truncated. show original

Thanks

ragav · October 27, 2016, 9:09am

Many thanks for guiding me @rohit_w . I will try and revert back if I am struck somewhere .

Thanks again

ragav · October 27, 2016, 9:43am

Hi , @rohit_w

I edited the dta.py file and added this

@frappe.whitelist()
def make_dta(source_name, target_doc=None):

doclist = get_mapped_doc("dta", source_name,		{
		"dta": {
			"doctype": "vave",
			
		},

return doclist

but I am getting an error now ,

Traceback (most recent call last):
  File "/home/ommi/frappe-bench/apps/frappe/frappe/app.py", line 55, in application
    response = frappe.handler.handle()
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 19, in handle
    execute_cmd(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 30, in execute_cmd
    method = get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 114, in get_attr
    method = frappe.get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/__init__.py", line 856, in get_attr
    return getattr(get_module(modulename), methodname)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/__init__.py", line 643, in get_module
    return importlib.import_module(modulename)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/home/ommi/frappe-bench/apps/vave/vave/vave/doctype/dta/dta.py", line 16
    doclist = get_mapped_doc("dta", source_name,		{
          ^
IndentationError: expected an indented block

Could you please help me ?

Thanks in advance

makarand_b · October 27, 2016, 10:08am

Its indentation error, check and correct indentation on line number 16

ragav · October 27, 2016, 11:03am

Thanks for the reply @makarand_b , I checked again everything seems to be correct , could you just indicate where I am wrong ?

Thanks in advance

makarand_b · October 27, 2016, 12:44pm

dta.py don’t have code in given repo, can you push your code ?

ragav · October 28, 2016, 3:57am

Many thanks for the reply @makarand_b . So kind of you for doing such a help , I pushed it to my repo

Many thanks for the support

makarand_b · October 28, 2016, 3:58am

@frappe.whitelist()
def make_dta(source_name, target_doc=None):

doclist = get_mapped_doc("dta", source_name,		{
		"dta": {
			"doctype": "vave",
			
		},

should be

@frappe.whitelist()
def make_dta(source_name, target_doc=None):
	doclist = get_mapped_doc("dta", source_name,{
			"dta": {
				"doctype": "vave",
				
			}, target_doc)
	return doclist

Thanks, Makarand

ragav · October 28, 2016, 4:00am

Thanks a ton @makarand_b for swift reply . You are awesome .

I will check , many thanks my friend

ragav · October 28, 2016, 4:04am

Hi @makarand_b ,

Nope again I am getting a syntax error , please help .

Traceback (most recent call last):
  File "/home/ommi/frappe-bench/apps/frappe/frappe/app.py", line 55, in application
    response = frappe.handler.handle()
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 19, in handle
    execute_cmd(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 30, in execute_cmd
    method = get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/handler.py", line 114, in get_attr
    method = frappe.get_attr(cmd)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/__init__.py", line 856, in get_attr
    return getattr(get_module(modulename), methodname)
  File "/home/ommi/frappe-bench/apps/frappe/frappe/__init__.py", line 643, in get_module
    return importlib.import_module(modulename)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/home/ommi/frappe-bench/apps/vave/vave/vave/doctype/dta/dta.py", line 20
    return doclist
         ^
SyntaxError: invalid syntax

Thanks in advance

makarand_b · October 28, 2016, 4:13am

add one more tab to return doclist line. for get_map_doc syntax please check the sales_order.py, sales_invoice.py files.

ragav · October 28, 2016, 4:19am

Thanks for the reply @makarand_b . I already did that , you could see that in my repo , dunno exactly what is causing this error , please guide me

Thanks