[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#695174: t-p-u pre-approval owncloud/4.0.4debian2-3.1



On Wed, Dec 12, 2012 at 19:44:38 +0000, Adam D. Barratt wrote:

> Control: tags -1 + moreinfo
> 
> On Mon, 2012-12-10 at 20:14 +0100, Julien Cristau wrote:
> > On Wed, Dec  5, 2012 at 22:18:54 +0100, Michael Banck wrote:
> > > 
> > > On Wed, Dec 05, 2012 at 10:55:56AM +0100, Julien Cristau wrote:
> > > > As far as I can tell this escapeHTML function is not defined in the
> > > > current version?  Upstream git has it in core/js/js.js.
> > > 
> > > Attached is a new candidate debdiff.
> > > 
> > Assuming this is tested, go ahead.
> 
> +Index: owncloud-4.0.4debian2/core/js/js.js
> [...]
> ++function escapeHTML(s) {
> [...]
> +Index: owncloud-4.0.4debian2/3rdparty/fullcalendar/js/fullcalendar.js
> [...]
> +-                              "<span class='fc-event-title'>" + event.title + "</span>" +
> ++                              "<span class='fc-event-title'>" + htmlEscape(event.title) + "</span>" +
> 
> Should the htmlEscape() call in that last hunk be escapeHTML()?
> 
iirc fullcalendar has its own preexisting escape function, with a
different name.

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


Reply to: