Opened 5 years ago

Closed 4 years ago

#1485475 closed Bugs (fixed)

htmlspecialchar on debian (and others)

Reported by: cebe Owned by:
Priority: 3 Milestone: 0.3-stable
Component: PHP backend Version: 0.2-beta
Severity: trivial Keywords:
Cc:

Description

Hi,

On PHP version previous to 5.2.3 (on debian stable, it's 5.2.0), htmlspecialchars only take 3 arguments, not 4 as it's write in rc code (roundcube/program/steps/mail/compose.inc on line 529).
Moreovers, the 4th argument is default value : no need to add it.
But if you really want, use phpversion() to check that.

Thanks.

Attachments (1)

rc_htmlspecialchars.diff​ (1.2 KB) - added by Erika 4 years ago.
Removal of (useless) 4th parameter of htmlspecialchars calls

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 years ago by till

  • Summary changed from htmlspeciachar on debian (and others) to htmlspecialchar on debian (and others)

comment:2 Changed 5 years ago by alec

  • Component changed from Client Scripts to PHP backend
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [27a12edc].

Changed 4 years ago by Erika

Removal of (useless) 4th parameter of htmlspecialchars calls

comment:3 Changed 4 years ago by Erika

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:4 Changed 4 years ago by alec

  • Milestone changed from 0.2-stable to 0.3-stable
  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in [3e8d8975].

Note: See TracTickets for help on using tickets.