blob: 9260d287852f4bc08c98f8787bd99f6fc593396d [file] [log] [blame]
# Copyright 2016 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style
# license that can be found in the LICENSE file or at
"""Servlet that implements the entry of new issues."""
from __future__ import print_function
from __future__ import division
from __future__ import absolute_import
import collections
import difflib
import logging
import string
import time
from businesslogic import work_env
from features import hotlist_helpers
from features import send_notifications
from framework import framework_bizobj
from framework import framework_constants
from framework import framework_helpers
from framework import framework_views
from framework import permissions
from framework import servlet
from framework import template_helpers
from framework import urls
from third_party import ezt
from tracker import field_helpers
from tracker import template_helpers as issue_tmpl_helpers
from tracker import tracker_bizobj
from tracker import tracker_constants
from tracker import tracker_helpers
from tracker import tracker_views
from proto import tracker_pb2
PLACEHOLDER_SUMMARY = 'Enter one-line summary'
PHASES_WITH_MILESTONES = ['Beta', 'Stable', 'Stable-Exp', 'Stable-Full']
CORP_RESTRICTION_LABEL = 'Restrict-View-Google'
class IssueEntry(servlet.Servlet):
"""IssueEntry shows a page with a simple form to enter a new issue."""
_PAGE_TEMPLATE = 'tracker/issue-entry-page.ezt'
def AssertBasePermission(self, mr):
"""Check whether the user has any permission to visit this page.
mr: commonly used info parsed from the request.
super(IssueEntry, self).AssertBasePermission(mr)
if not self.CheckPerm(mr, permissions.CREATE_ISSUE):
raise permissions.PermissionException(
'User is not allowed to enter an issue')
def GatherPageData(self, mr):
"""Build up a dictionary of data values to use when rendering the page.
mr: commonly used info parsed from the request.
Dict of values used by EZT for rendering the page.
with mr.profiler.Phase('getting config'):
config =, mr.project_id)
# In addition to checking perms, we adjust some default field values for
# project members.
is_member = framework_bizobj.UserIsInProject(
mr.project, mr.auth.effective_ids)
page_perms = self.MakePagePerms(
mr, None,
with work_env.WorkEnv(mr, as we:
userprefs = we.GetUserPrefs(mr.auth.user_id)
code_font = any(pref for pref in userprefs.prefs
if == 'code_font' and pref.value == 'true')
template = self._GetTemplate(mr.cnxn, config, mr.template_name, is_member)
if template.summary:
initial_summary = template.summary
initial_summary_must_be_edited = template.summary_must_be_edited
initial_summary = PLACEHOLDER_SUMMARY
initial_summary_must_be_edited = True
if template.status:
initial_status = template.status
elif is_member:
initial_status = 'Accepted'
initial_status = 'New' # not offering meta, only used in hidden field.
component_paths = []
for component_id in template.component_ids:
tracker_bizobj.FindComponentDefByID(component_id, config).path)
initial_components = ', '.join(component_paths)
if template.owner_id:
initial_owner = framework_views.MakeUserView(
mr.cnxn,, template.owner_id)
elif template.owner_defaults_to_member and page_perms.EditIssue:
initial_owner = mr.auth.user_view
initial_owner = None
if initial_owner:
initial_owner_name =
owner_avail_state = initial_owner.avail_state
owner_avail_message_short = initial_owner.avail_message_short
initial_owner_name = ''
owner_avail_state = None
owner_avail_message_short = None
# Check whether to allow attachments from the entry page
allow_attachments = tracker_helpers.IsUnderSoftAttachmentQuota(mr.project)
config_view = tracker_views.ConfigView(mr,, config, template)
# If the user followed a link that specified the template name, make sure
# that it is also in the menu as the current choice.
# TODO(jeffcarp): Unit test this.
config_view.template_view.can_view = ezt.boolean(True)
# TODO(jeffcarp): Unit test this.
offer_templates = len(config_view.template_names) > 1
restrict_to_known = config.restrict_to_known
enum_field_name_set = {fd.field_name.lower() for fd in config.field_defs
if fd.field_type is tracker_pb2.FieldTypes.ENUM_TYPE
and not fd.is_deleted} # TODO(jrobbins): restrictions
link_or_template_labels = mr.GetListParam('labels', template.labels)
labels = [lab for lab in link_or_template_labels
if not tracker_bizobj.LabelIsMaskedByField(
lab, enum_field_name_set)]
# Corp-mode users automatically add R-V-G.
with work_env.WorkEnv(mr, as we:
userprefs = we.GetUserPrefs(mr.auth.user_id)
corp_mode = any( == 'restrict_new_issues' and up.value == 'true'
for up in userprefs.prefs)
if corp_mode:
if not any(lab.lower().startswith('restrict-view-') for lab in labels):
field_user_views = tracker_views.MakeFieldUserViews(
mr.cnxn, template,
approval_ids = [av.approval_id for av in template.approval_values]
field_views = tracker_views.MakeAllFieldValueViews(
config, link_or_template_labels, [], template.field_values,
field_user_views, parent_approval_ids=approval_ids,
# TODO(jrobbins): remove "or []" after next release.
(prechecked_approvals, required_approval_ids,
phases) = issue_tmpl_helpers.GatherApprovalsPageData(
template.approval_values or [], template.phases, config)
approvals = [view for view in field_views if view.field_id in
page_data = {
'issue_tab_mode': 'issueEntry',
'initial_summary': initial_summary,
'template_summary': initial_summary,
'clear_summary_on_click': ezt.boolean(
initial_summary_must_be_edited and
'initial_summary' not in mr.form_overrides),
'must_edit_summary': ezt.boolean(initial_summary_must_be_edited),
'initial_description': template.content,
'component_required': ezt.boolean(template.component_required),
'initial_status': initial_status,
'initial_owner': initial_owner_name,
'owner_avail_state': owner_avail_state,
'owner_avail_message_short': owner_avail_message_short,
'initial_components': initial_components,
'initial_cc': '',
'initial_blocked_on': '',
'initial_blocking': '',
'initial_hotlists': '',
'labels': labels,
'fields': field_views,
'any_errors': ezt.boolean(mr.errors.AnyErrors()),
'page_perms': page_perms,
'allow_attachments': ezt.boolean(allow_attachments),
'max_attach_size': template_helpers.BytesKbOrMb(
'offer_templates': ezt.boolean(offer_templates),
'config': config_view,
'restrict_to_known': ezt.boolean(restrict_to_known),
'is_member': ezt.boolean(is_member),
'code_font': ezt.boolean(code_font),
# The following are necessary for displaying phases that come with
# this template. These are read-only.
'allow_edit': ezt.boolean(False),
'initial_phases': phases,
'approvals': approvals,
'prechecked_approvals': prechecked_approvals,
'required_approval_ids': required_approval_ids,
# See monorail:4692 and the use of PHASES_WITH_MILESTONES
# in elements/flt/mr-launch-overview/mr-phase.js
'issue_phase_names': list(
{ for phase in phases if
return page_data
def GatherHelpData(self, mr, page_data):
"""Return a dict of values to drive on-page user help.
mr: commonly used info parsed from the request.
page_data: Dictionary of base and page template data.
A dict of values to drive on-page user help, to be added to page_data.
help_data = super(IssueEntry, self).GatherHelpData(mr, page_data)
dismissed = []
if mr.auth.user_pb:
with work_env.WorkEnv(mr, as we:
userprefs = we.GetUserPrefs(mr.auth.user_id)
dismissed = [ for pv in userprefs.prefs if pv.value == 'true']
is_privileged_domain_user = framework_bizobj.IsPriviledgedDomainUser(
if (mr.auth.user_id and
'privacy_click_through' not in dismissed):
help_data['cue'] = 'privacy_click_through'
elif (mr.auth.user_id and
'code_of_conduct' not in dismissed):
help_data['cue'] = 'code_of_conduct'
'is_privileged_domain_user': ezt.boolean(is_privileged_domain_user),
return help_data
def ProcessFormData(self, mr, post_data):
"""Process the issue entry form.
mr: commonly used info parsed from the request.
post_data: The post_data dict for the current request.
String URL to redirect the user to after processing.
config =, mr.project_id)
parsed = tracker_helpers.ParseIssueRequest(
mr.cnxn, post_data,, mr.errors, mr.project_name)
bounce_labels = parsed.labels[:]
bounce_fields = tracker_views.MakeBounceFieldValueViews(
parsed.fields.vals, parsed.fields.phase_vals, config)
parsed.labels, parsed.labels_remove, parsed.fields.vals,
parsed.fields.vals_remove, config)
is_member = framework_bizobj.UserIsInProject(
mr.project, mr.auth.effective_ids)
template = self._GetTemplate(
mr.cnxn, config, parsed.template_name, is_member)
phases) = issue_tmpl_helpers.FilterApprovalsAndPhases(
template.approval_values or [], template.phases, config)
phase_ids_by_name = { [phase.phase_id] for phase in template.phases}
field_values = field_helpers.ParseFieldValues(
mr.cnxn,, parsed.fields.vals,
parsed.fields.phase_vals, config,
labels = _DiscardUnusedTemplateLabelPrefixes(parsed.labels)
component_ids = tracker_helpers.LookupComponentIDs(
parsed.components.paths, config, mr.errors)
# TODO(jrobbins): consider captcha 3 score in API
if not parsed.summary.strip() or parsed.summary == PLACEHOLDER_SUMMARY:
mr.errors.summary = 'Summary is required'
if not parsed.comment.strip():
mr.errors.comment = 'A description is required'
if len(parsed.comment) > tracker_constants.MAX_COMMENT_CHARS:
mr.errors.comment = 'Comment is too long'
if len(parsed.summary) > tracker_constants.MAX_SUMMARY_CHARS:
mr.errors.summary = 'Summary is too long'
if _MatchesTemplate(parsed.comment, template):
mr.errors.comment = 'Template must be filled out.'
if parsed.users.owner_id is None:
mr.errors.owner = 'Invalid owner username'
valid, msg = tracker_helpers.IsValidIssueOwner(
mr.cnxn, mr.project, parsed.users.owner_id,
if not valid:
mr.errors.owner = msg
if None in parsed.users.cc_ids: = 'Invalid Cc username'
mr,, field_values, config, mr.errors)
hotlist_pbs = ProcessParsedHotlistRefs(
mr,, parsed.hotlists.hotlist_refs)
if not mr.errors.AnyErrors():
with work_env.WorkEnv(mr, as we:
if parsed.attachments:
new_bytes_used = tracker_helpers.ComputeNewQuotaBytesUsed(
mr.project, parsed.attachments)
# TODO(jrobbins): Make quota be calculated and stored as
# part of applying the comment.
mr.cnxn, mr.project.project_id,
marked_description = tracker_helpers.MarkupDescriptionOnInput(
parsed.comment, template.content)
has_star = 'star' in post_data and post_data['star'] == '1'
if approval_values:
_AttachDefaultApprovers(config, approval_values)
issue, _ = we.CreateIssue(
mr.project_id, parsed.summary, parsed.status,
parsed.users.owner_id, parsed.users.cc_ids, labels, field_values,
component_ids, marked_description,
blocking=parsed.blocking.iids, attachments=parsed.attachments,
approval_values=approval_values, phases=phases)
if has_star:
we.StarIssue(issue, True)
if hotlist_pbs:
hotlist_ids = {hotlist.hotlist_id for hotlist in hotlist_pbs}
issue_tuple = (issue.issue_id, mr.auth.user_id, int(time.time()),
mr.cnxn, hotlist_ids, issue_tuple,,
except tracker_helpers.OverAttachmentQuota:
mr.errors.attachments = 'Project attachment quota exceeded.'
mr.template_name = parsed.template_name
if mr.errors.AnyErrors():
mr, initial_summary=parsed.summary, initial_status=parsed.status,
initial_cc=', '.join(parsed.users.cc_usernames),
initial_components=', '.join(parsed.components.paths),
initial_comment=parsed.comment, labels=bounce_labels,
fields=bounce_fields, template_name=parsed.template_name,
# format a redirect url
return framework_helpers.FormatAbsoluteURL(
mr, urls.ISSUE_DETAIL, id=issue.local_id)
def _GetTemplate(self, cnxn, config, template_name, is_member):
"""Tries to fetch template by name and implements default template logic
if not found."""
template = None
if template_name:
template_name = template_name.replace('+', ' ')
template =,
template_name, config.project_id)
if not template:
if is_member:
template_id = config.default_template_for_developers
template_id = config.default_template_for_users
template =, template_id)
# If the default templates were deleted, load all and pick the first one.
if not template:
templates =,
assert len(templates) > 0, 'Project has no templates!'
template = templates[0]
return template
def _AttachDefaultApprovers(config, approval_values):
approval_defs_by_id = {ad.approval_id: ad for ad in config.approval_defs}
for av in approval_values:
ad = approval_defs_by_id.get(av.approval_id)
if ad:
av.approver_ids = ad.approver_ids[:]
else:'ApprovalDef with approval_id %r could not be found',
def _MatchesTemplate(content, template):
content = content.strip(string.whitespace)
template_content = template.content.strip(string.whitespace)
diff = difflib.unified_diff(content.splitlines(),
return len('\n'.join(diff)) == 0
def _DiscardUnusedTemplateLabelPrefixes(labels):
"""Drop any labels that end in '-?'.
labels: a list of label strings.
A list of the same labels, but without any that end with '-?'.
Those label prefixes in the new issue templates are intended to
prompt the user to enter some label with that prefix, but if
nothing is entered there, we do not store anything.
return [lab for lab in labels
if not lab.endswith('-?')]
def ProcessParsedHotlistRefs(mr, services, parsed_hotlist_refs):
"""Process a list of ParsedHotlistRefs, returning referenced hotlists.
This function validates the given ParsedHotlistRefs using four checks; if all
of them succeed, then it returns the corresponding hotlist protobuf objects.
If any of them fail, it sets the appropriate error string in mr.errors, and
returns an empty list.
mr: the MonorailRequest object
services: the service manager
parsed_hotlist_refs: a list of ParsedHotlistRef objects
on valid input, a list of hotlist protobuf objects
if a check fails (and the input is thus considered invalid), an empty list
if any of the checks fails, set mr.errors.hotlists to a descriptive error
# Pre-processing; common pieces used by functions later.
user_hotlist_pbs = services.features.GetHotlistsByUserID(
mr.cnxn, mr.auth.user_id)
user_hotlist_owners_ids = {hotlist.owner_ids[0]
for hotlist in user_hotlist_pbs}
user_hotlist_owners_to_emails = services.user.LookupUserEmails(
mr.cnxn, user_hotlist_owners_ids)
user_hotlist_emails_to_owners = {v: k
for k, v in user_hotlist_owners_to_emails.items()}
user_hotlist_refs_to_pbs = {
hotlist_helpers.HotlistRef(hotlist.owner_ids[0], hotlist
for hotlist in user_hotlist_pbs }
short_refs = list()
full_refs = list()
for parsed_ref in parsed_hotlist_refs:
if parsed_ref.user_email is None:
invalid_names = hotlist_helpers.InvalidParsedHotlistRefsNames(
parsed_hotlist_refs, user_hotlist_pbs)
if invalid_names:
mr.errors.hotlists = (
'You have no hotlist(s) named: %s' % ', '.join(invalid_names))
return []
ambiguous_names = hotlist_helpers.AmbiguousShortrefHotlistNames(
short_refs, user_hotlist_pbs)
if ambiguous_names:
mr.errors.hotlists = (
'Ambiguous hotlist(s) specified: %s' % ', '.join(ambiguous_names))
return []
# At this point, all refs' named hotlists are guaranteed to exist, and
# short refs are guaranteed to be unambiguous;
# therefore, short refs are also valid.
short_refs_hotlist_names = {sref.hotlist_name for sref in short_refs}
shortref_valid_pbs = [hotlist for hotlist in user_hotlist_pbs
if in short_refs_hotlist_names]
invalid_emails = hotlist_helpers.InvalidParsedHotlistRefsEmails(
full_refs, user_hotlist_emails_to_owners)
if invalid_emails:
mr.errors.hotlists = (
'You have no hotlist(s) owned by: %s' % ', '.join(invalid_emails))
return []
fullref_valid_pbs, invalid_refs = (
full_refs, user_hotlist_emails_to_owners, user_hotlist_refs_to_pbs))
if invalid_refs:
invalid_refs_readable = [':'.join(parsed_ref)
for parsed_ref in invalid_refs]
mr.errors.hotlists = (
'Not in your hotlist(s): %s' % ', '.join(invalid_refs_readable))
return []
hotlist_pbs = shortref_valid_pbs + fullref_valid_pbs
return hotlist_pbs