From dee4a7336c56a36818ce79b48eff787bb66b7ae0 Mon Sep 17 00:00:00 2001 From: Suraj Shetty Date: Wed, 29 Jan 2020 15:22:35 +0530 Subject: [PATCH] fix(translations): Incorrect syntax --- frappe/__init__.py | 2 +- .../automation/doctype/assignment_rule/assignment_rule.py | 2 +- frappe/client.py | 2 +- .../report/addresses_and_contacts/addresses_and_contacts.py | 2 +- frappe/core/doctype/data_import/importer.py | 2 +- frappe/core/doctype/doctype/doctype.py | 2 +- frappe/core/doctype/user/user.py | 2 +- frappe/core/doctype/user_permission/user_permission.py | 2 +- frappe/desk/doctype/todo/todo.py | 2 +- frappe/desk/form/assign_to.py | 2 +- frappe/desk/page/setup_wizard/setup_wizard.py | 3 ++- frappe/desk/query_report.py | 2 +- frappe/email/doctype/auto_email_report/auto_email_report.py | 4 ++-- frappe/email/doctype/email_account/email_account.py | 6 +++--- frappe/integrations/doctype/oauth_client/oauth_client.py | 2 +- frappe/integrations/utils.py | 4 ++-- frappe/model/base_document.py | 2 +- frappe/model/document.py | 4 ++-- frappe/model/workflow.py | 2 +- frappe/public/js/frappe/desk.js | 2 +- frappe/share.py | 2 +- frappe/templates/emails/auto_email_report.html | 2 +- frappe/utils/change_log.py | 2 +- frappe/workflow/doctype/workflow_action/workflow_action.py | 6 +++--- frappe/www/confirm_workflow_action.html | 2 +- frappe/www/third_party_apps.html | 2 +- 26 files changed, 34 insertions(+), 33 deletions(-) diff --git a/frappe/__init__.py b/frappe/__init__.py index 01187541d3..12e5204001 100644 --- a/frappe/__init__.py +++ b/frappe/__init__.py @@ -544,7 +544,7 @@ def only_for(roles, message=False): myroles = set(get_roles()) if not roles.intersection(myroles): if message: - msgprint(_('Only for {}'.format(', '.join(roles)))) + msgprint(_('Only for {}').format(', '.join(roles))) raise PermissionError def get_domain_data(module): diff --git a/frappe/automation/doctype/assignment_rule/assignment_rule.py b/frappe/automation/doctype/assignment_rule/assignment_rule.py index 55792b2648..5d8dab90ce 100644 --- a/frappe/automation/doctype/assignment_rule/assignment_rule.py +++ b/frappe/automation/doctype/assignment_rule/assignment_rule.py @@ -16,7 +16,7 @@ class AssignmentRule(Document): assignment_days = self.get_assignment_days() if not len(set(assignment_days)) == len(assignment_days): repeated_days = get_repeated(assignment_days) - frappe.throw(_("Assignment Day {0} has been repeated.".format(frappe.bold(repeated_days)))) + frappe.throw(_("Assignment Day {0} has been repeated.").format(frappe.bold(repeated_days))) def on_update(self): # pylint: disable=no-self-use frappe.cache_manager.clear_doctype_map('Assignment Rule', self.name) diff --git a/frappe/client.py b/frappe/client.py index 9f42869347..045e28dd8f 100644 --- a/frappe/client.py +++ b/frappe/client.py @@ -70,7 +70,7 @@ def get_value(doctype, fieldname, filters=None, as_dict=True, debug=False, paren check_parent_permission(parent, doctype) if not frappe.has_permission(doctype): - frappe.throw(_("No permission for {0}".format(doctype)), frappe.PermissionError) + frappe.throw(_("No permission for {0}").format(doctype), frappe.PermissionError) filters = get_safe_filters(filters) diff --git a/frappe/contacts/report/addresses_and_contacts/addresses_and_contacts.py b/frappe/contacts/report/addresses_and_contacts/addresses_and_contacts.py index ba0ecd41d1..1b3982f251 100644 --- a/frappe/contacts/report/addresses_and_contacts/addresses_and_contacts.py +++ b/frappe/contacts/report/addresses_and_contacts/addresses_and_contacts.py @@ -95,7 +95,7 @@ def get_reference_details(reference_doctype, doctype, reference_list, reference_ temp_records.append(d[1:]) if not reference_list: - frappe.throw(_("No records present in {0}".format(reference_doctype))) + frappe.throw(_("No records present in {0}").format(reference_doctype)) reference_details[reference_list[0]][frappe.scrub(doctype)] = temp_records return reference_details diff --git a/frappe/core/doctype/data_import/importer.py b/frappe/core/doctype/data_import/importer.py index f3c23cbfea..b6d410d072 100644 --- a/frappe/core/doctype/data_import/importer.py +++ b/frappe/core/doctype/data_import/importer.py @@ -222,7 +222,7 @@ def upload(rows = None, submit_after_import=None, ignore_encoding_errors=False, if (autoname not in doc) or (not doc[autoname]): from frappe.model.base_document import get_controller if not hasattr(get_controller(doctype), "autoname"): - frappe.throw(_("{0} is a mandatory field".format(autoname))) + frappe.throw(_("{0} is a mandatory field").format(autoname)) return True users = frappe.db.sql_list("select name from tabUser") diff --git a/frappe/core/doctype/doctype/doctype.py b/frappe/core/doctype/doctype/doctype.py index de8221da29..da1b184cc1 100644 --- a/frappe/core/doctype/doctype/doctype.py +++ b/frappe/core/doctype/doctype/doctype.py @@ -247,7 +247,7 @@ class DocType(Document): if autoname and autoname.startswith('field:'): field = autoname.split(":")[1] if not field or field not in [ df.fieldname for df in self.fields ]: - frappe.throw(_("Invalid fieldname '{0}' in autoname".format(field))) + frappe.throw(_("Invalid fieldname '{0}' in autoname").format(field)) else: for df in self.fields: if df.fieldname == field: diff --git a/frappe/core/doctype/user/user.py b/frappe/core/doctype/user/user.py index 5dfa3b0ace..a4f546f701 100644 --- a/frappe/core/doctype/user/user.py +++ b/frappe/core/doctype/user/user.py @@ -262,7 +262,7 @@ class User(Document): if not subject: site_name = frappe.db.get_default('site_name') or frappe.get_conf().get("site_name") if site_name: - subject = _("Welcome to {0}".format(site_name)) + subject = _("Welcome to {0}").format(site_name) else: subject = _("Complete Registration") diff --git a/frappe/core/doctype/user_permission/user_permission.py b/frappe/core/doctype/user_permission/user_permission.py index 7be915cbec..48d4fcb5d4 100644 --- a/frappe/core/doctype/user_permission/user_permission.py +++ b/frappe/core/doctype/user_permission/user_permission.py @@ -53,7 +53,7 @@ class UserPermission(Document): }, limit=1) if overlap_exists: ref_link = frappe.get_desk_link(self.doctype, overlap_exists[0].name) - frappe.throw(_("{0} has already assigned default value for {1}.".format(ref_link, self.allow))) + frappe.throw(_("{0} has already assigned default value for {1}.").format(ref_link, self.allow)) @frappe.whitelist() def get_user_permissions(user=None): diff --git a/frappe/desk/doctype/todo/todo.py b/frappe/desk/doctype/todo/todo.py index 5d04f412c0..6cd7c68368 100644 --- a/frappe/desk/doctype/todo/todo.py +++ b/frappe/desk/doctype/todo/todo.py @@ -31,7 +31,7 @@ class ToDo(Document): # NOTE the previous value is only available in validate method if self.get_db_value("status") != self.status: self._assignment = { - "text": frappe._("Assignment closed by {0}".format(get_fullname(frappe.session.user))), + "text": frappe._("Assignment closed by {0}").format(get_fullname(frappe.session.user)), "comment_type": "Assignment Completed" } diff --git a/frappe/desk/form/assign_to.py b/frappe/desk/form/assign_to.py index 9714f31d1f..76c7caa63d 100644 --- a/frappe/desk/form/assign_to.py +++ b/frappe/desk/form/assign_to.py @@ -51,7 +51,7 @@ def add(args=None): from frappe.utils import nowdate if not args.get('description'): - args['description'] = _('Assignment for {0} {1}'.format(args['doctype'], args['name'])) + args['description'] = _('Assignment for {0} {1}').format(args['doctype'], args['name']) d = frappe.get_doc({ "doctype":"ToDo", diff --git a/frappe/desk/page/setup_wizard/setup_wizard.py b/frappe/desk/page/setup_wizard/setup_wizard.py index 43d4e8dde4..b297d13bb2 100755 --- a/frappe/desk/page/setup_wizard/setup_wizard.py +++ b/frappe/desk/page/setup_wizard/setup_wizard.py @@ -69,7 +69,8 @@ def setup_complete(args): for task in stage.get('tasks'): current_task = task task.get('fn')(task.get('args')) - except Exception: + except Exception as e: + print(e) handle_setup_exception(args) return {'status': 'fail', 'fail': current_task.get('fail_msg')} else: diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index 99e3c2e96f..2c5025b1cf 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -444,7 +444,7 @@ def save_report(reference_report, report_name, columns): 'report_type': 'Custom Report', 'reference_report': reference_report }).insert(ignore_permissions = True) - frappe.msgprint(_("{0} saved successfully".format(new_report.name))) + frappe.msgprint(_("{0} saved successfully").format(new_report.name)) return new_report.name diff --git a/frappe/email/doctype/auto_email_report/auto_email_report.py b/frappe/email/doctype/auto_email_report/auto_email_report.py index fcf32640fc..c729795c82 100644 --- a/frappe/email/doctype/auto_email_report/auto_email_report.py +++ b/frappe/email/doctype/auto_email_report/auto_email_report.py @@ -53,8 +53,8 @@ class AutoEmailReport(Document): """ check if user has select correct report format """ valid_report_formats = ["HTML", "XLSX", "CSV"] if self.format not in valid_report_formats: - frappe.throw(_("%s is not a valid report format. Report format should \ - one of the following %s"%(frappe.bold(self.format), frappe.bold(", ".join(valid_report_formats))))) + frappe.throw(_("{0} is not a valid report format. Report format should one of the following {1}" \ + .format(frappe.bold(self.format), frappe.bold(", ".join(valid_report_formats))))) def get_report_content(self): '''Returns file in for the report in given format''' diff --git a/frappe/email/doctype/email_account/email_account.py b/frappe/email/doctype/email_account/email_account.py index 41b47a1fa2..f3eb2188b7 100755 --- a/frappe/email/doctype/email_account/email_account.py +++ b/frappe/email/doctype/email_account/email_account.py @@ -52,8 +52,8 @@ class EmailAccount(Document): "name": ("!=", self.name) }) if duplicate_email_account: - frappe.throw(_("Email ID must be unique, Email Account already exists \ - for {0}".format(frappe.bold(self.email_id)))) + frappe.throw(_("Email ID must be unique, Email Account already exists for {0}") \ + .format(frappe.bold(self.email_id))) if frappe.local.flags.in_patch or frappe.local.flags.in_test: return @@ -175,7 +175,7 @@ class EmailAccount(Document): # if called via self.receive and it leads to authentication error, disable incoming # and send email to system manager self.handle_incoming_connect_error( - description=_('Authentication failed while receiving emails from Email Account {0}. Message from server: {1}'.format(self.name, e.message)) + description=_('Authentication failed while receiving emails from Email Account {0}. Message from server: {1}').format(self.name, e.message) ) return None diff --git a/frappe/integrations/doctype/oauth_client/oauth_client.py b/frappe/integrations/doctype/oauth_client/oauth_client.py index 2663623bfa..02f5041dfb 100644 --- a/frappe/integrations/doctype/oauth_client/oauth_client.py +++ b/frappe/integrations/doctype/oauth_client/oauth_client.py @@ -16,4 +16,4 @@ class OAuthClient(Document): def validate_grant_and_response(self): if self.grant_type == "Authorization Code" and self.response_type != "Code" or \ self.grant_type == "Implicit" and self.response_type != "Token": - frappe.throw(_("Combination of Grant Type ({0}) and Response Type ({1}) not allowed".format(self.grant_type, self.response_type))) + frappe.throw(_("Combination of Grant Type ({0}) and Response Type ({1}) not allowed").format(self.grant_type, self.response_type)) diff --git a/frappe/integrations/utils.py b/frappe/integrations/utils.py index ca4c7bc3cb..811b007131 100644 --- a/frappe/integrations/utils.py +++ b/frappe/integrations/utils.py @@ -77,12 +77,12 @@ def get_payment_gateway_controller(payment_gateway): try: return frappe.get_doc("{0} Settings".format(payment_gateway)) except Exception: - frappe.throw(_("{0} Settings not found".format(payment_gateway))) + frappe.throw(_("{0} Settings not found").format(payment_gateway)) else: try: return frappe.get_doc(gateway.gateway_settings, gateway.gateway_controller) except Exception: - frappe.throw(_("{0} Settings not found".format(payment_gateway))) + frappe.throw(_("{0} Settings not found").format(payment_gateway)) @frappe.whitelist(allow_guest=True, xss_safe=True) diff --git a/frappe/model/base_document.py b/frappe/model/base_document.py index 28ac740580..5ab31cdf66 100644 --- a/frappe/model/base_document.py +++ b/frappe/model/base_document.py @@ -392,7 +392,7 @@ class BaseDocument(object): if df: label = df.label - frappe.msgprint(_("{0} must be unique".format(label or fieldname))) + frappe.msgprint(_("{0} must be unique").format(label or fieldname)) # this is used to preserve traceback raise frappe.UniqueValidationError(self.doctype, self.name, e) diff --git a/frappe/model/document.py b/frappe/model/document.py index 134cce3b75..68a65e0401 100644 --- a/frappe/model/document.py +++ b/frappe/model/document.py @@ -1115,9 +1115,9 @@ class Document(BaseDocument): label = doc.meta.get_label(fieldname) condition_str = error_condition_map.get(condition, condition) if doc.parentfield: - msg = _("Incorrect value in row {0}: {1} must be {2} {3}".format(doc.idx, label, condition_str, val2)) + msg = _("Incorrect value in row {0}: {1} must be {2} {3}").format(doc.idx, label, condition_str, val2) else: - msg = _("Incorrect value: {0} must be {1} {2}".format(label, condition_str, val2)) + msg = _("Incorrect value: {0} must be {1} {2}").format(label, condition_str, val2) # raise passed exception or True msgprint(msg, raise_exception=raise_exception or True) diff --git a/frappe/model/workflow.py b/frappe/model/workflow.py index 548d713e6f..647812014a 100644 --- a/frappe/model/workflow.py +++ b/frappe/model/workflow.py @@ -139,7 +139,7 @@ def validate_workflow(doc): state_row = [d for d in workflow.states if d.state == current_state] if not state_row: - frappe.throw(_('{0} is not a valid Workflow State. Please update your Workflow and try again.'.format(frappe.bold(current_state)))) + frappe.throw(_('{0} is not a valid Workflow State. Please update your Workflow and try again.').format(frappe.bold(current_state))) state_row = state_row[0] # if transitioning, check if user is allowed to transition diff --git a/frappe/public/js/frappe/desk.js b/frappe/public/js/frappe/desk.js index 5e642f907b..6a9b3a0b09 100644 --- a/frappe/public/js/frappe/desk.js +++ b/frappe/public/js/frappe/desk.js @@ -176,7 +176,7 @@ frappe.Application = Class.extend({ email_password_prompt: function(email_account,user,i) { var me = this; var d = new frappe.ui.Dialog({ - title: __('Email Account setup please enter your password for: '+email_account[i]["email_id"]), + title: __('Email Account setup please enter your password for: {0}', [email_account[i]["email_id"]]), fields: [ { 'fieldname': 'password', 'fieldtype': 'Password', diff --git a/frappe/share.py b/frappe/share.py index 03311bed4a..497e2056d5 100644 --- a/frappe/share.py +++ b/frappe/share.py @@ -145,7 +145,7 @@ def get_share_name(doctype, name, user, everyone): def check_share_permission(doctype, name): """Check if the user can share with other users""" if not frappe.has_permission(doctype, ptype="share", doc=name): - frappe.throw(_("No permission to {0} {1} {2}".format("share", doctype, name)), frappe.PermissionError) + frappe.throw(_("No permission to {0} {1} {2}").format("share", doctype, name), frappe.PermissionError) def notify_assignment(shared_by, doctype, doc_name, everyone, notify=0): diff --git a/frappe/templates/emails/auto_email_report.html b/frappe/templates/emails/auto_email_report.html index a658c988a9..129fb75cbd 100644 --- a/frappe/templates/emails/auto_email_report.html +++ b/frappe/templates/emails/auto_email_report.html @@ -54,7 +54,7 @@

- {{ _("This report was generated on {0}".format(date_time)) }} + {{ _("This report was generated on {0}").format(date_time) }}

{{ _("View report in your browser") }}: diff --git a/frappe/utils/change_log.py b/frappe/utils/change_log.py index 39be5db5c8..c94a247796 100644 --- a/frappe/utils/change_log.py +++ b/frappe/utils/change_log.py @@ -240,7 +240,7 @@ def show_update_popup(): title = app.title ) if release_links: - update_message += _("New {} releases for the following apps are available".format(update_type)) + ":

{}".format(release_links) + update_message += _("New {} releases for the following apps are available").format(update_type) + ":

{}".format(release_links) if update_message: frappe.msgprint(update_message, title=_("New updates are available"), indicator='green') diff --git a/frappe/workflow/doctype/workflow_action/workflow_action.py b/frappe/workflow/doctype/workflow_action/workflow_action.py index f32b1dfc03..ecccce7162 100644 --- a/frappe/workflow/doctype/workflow_action/workflow_action.py +++ b/frappe/workflow/doctype/workflow_action/workflow_action.py @@ -125,12 +125,12 @@ def return_action_confirmation_page(doc, action, action_link, alert_doc_change=F def return_link_expired_page(doc, doc_workflow_state): frappe.respond_as_web_page(_("Link Expired"), - _("Document {0} has been set to state {1} by {2}" + _("Document {0} has been set to state {1} by {2}") .format( frappe.bold(doc.get('name')), frappe.bold(doc_workflow_state), frappe.bold(frappe.get_value('User', doc.get("modified_by"), 'full_name')) - )), indicator_color='blue') + ), indicator_color='blue') def clear_old_workflow_actions(doc, user=None): user = user if user else frappe.session.user @@ -276,7 +276,7 @@ def get_common_email_args(doc): response = frappe.render_template(email_template.response, vars(doc)) else: subject = _('Workflow Action') - response = _('{0}: {1}'.format(doctype, docname)) + response = _('{0}: {1}').format(doctype, docname) common_args = { 'template': 'workflow_action', diff --git a/frappe/www/confirm_workflow_action.html b/frappe/www/confirm_workflow_action.html index 5f30a21676..5e6d97c153 100644 --- a/frappe/www/confirm_workflow_action.html +++ b/frappe/www/confirm_workflow_action.html @@ -1,7 +1,7 @@ {% extends "frappe/www/message.html" %} {% block message_body %}

- {{ _("Please confirm your action to {0} this document.".format(action)) }} + {{ _("Please confirm your action to {0} this document.").format(action) }}

{% if alert_doc_change %}

diff --git a/frappe/www/third_party_apps.html b/frappe/www/third_party_apps.html index b4eb6036de..d2a2e9d266 100644 --- a/frappe/www/third_party_apps.html +++ b/frappe/www/third_party_apps.html @@ -24,7 +24,7 @@

-
{{ _("This will log out {0} from all other devices".format(app.app_name)) }}
+
{{ _("This will log out {0} from all other devices").format(app.app_name) }}