Code review comment for lp:~thumper/launchpad/descriptions-for-merge-proposals

Revision history for this message
Paul Hummer (rockstar) wrote :

206 @property
207 + def description_html(self):
208 + """The description as widget HTML."""
209 + description = self.context.description
210 + if description is None:
211 + description = ''
212 + formatter = FormattersAPI
213 + hide_email = formatter(description).obfuscate_email()
214 + description = formatter(hide_email).text_to_html()
215 + return TextAreaEditorWidget(
216 + self.context,
217 + 'description',
218 + canonical_url(self.context, view_name='+edit-description'),
219 + id="edit-description",
220 + title="Description of the Change",
221 + value=description,
222 + accept_empty=True)
223 +
224 + @property

I don't particularly like this, but it's not an issue that you really need to worry about. I'll bring it up in the next reviewer meeting.

review: Approve (code js)

« Back to merge proposal