Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: cms/translations/xtm/utils.py

Issue 29908581: Issue #7036 - [XTM Integration] Add support for providing workflow name (Closed)
Left Patch Set: Created Oct. 12, 2018, 2:03 p.m.
Right Patch Set: Addressed comments from Patch Set 1 Created Oct. 18, 2018, 4:23 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
LEFTRIGHT
1 # This file is part of the Adblock Plus web scripts, 1 # This file is part of the Adblock Plus web scripts,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # GNU General Public License for more details. 11 # GNU General Public License for more details.
12 # 12 #
13 # You should have received a copy of the GNU General Public License 13 # You should have received a copy of the GNU General Public License
14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
15
16 from __future__ import unicode_literals
15 17
16 import collections 18 import collections
17 import logging 19 import logging
18 import os 20 import os
19 import time 21 import time
20 import json 22 import json
21 23
22 from cms.utils import process_page 24 from cms.utils import process_page
23 import cms.translations.xtm.constants as const 25 import cms.translations.xtm.constants as const
24 from cms.translations.xtm.xtm_api import XTMCloudException 26 from cms.translations.xtm.xtm_api import XTMCloudException
(...skipping 453 matching lines...) Expand 10 before | Expand all | Expand 10 after
478 if args.workflow_id: 480 if args.workflow_id:
479 return args.workflow_id 481 return args.workflow_id
480 482
481 possible_ids = api.get_workflows_by_name(args.workflow_name) 483 possible_ids = api.get_workflows_by_name(args.workflow_name)
482 484
483 if len(possible_ids) == 0: 485 if len(possible_ids) == 0:
484 raise Exception(const.ErrorMessages.NO_WORKFLOW_FOR_NAME.format( 486 raise Exception(const.ErrorMessages.NO_WORKFLOW_FOR_NAME.format(
485 args.workflow_name)) 487 args.workflow_name))
486 488
487 return possible_ids[0] 489 return possible_ids[0]
LEFTRIGHT

Powered by Google App Engine
This is Rietveld