developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#488 closed bug (fixed)

jResponseZip : addHttpHeaders don't exist

Reported by: bastnic Owned by: bastnic
Priority: high Milestone: Jelix 1.0.3
Component: jelix:core response Version: 1.0.2
Severity: critical Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

make call to a $this->addHttpHeaders()function which don't exist. The correct function's name is $this->addHttpHeader() (without 's' at the end of 'Header'.

Attachments (1)

patch-488.diff​ (1.4 KB) - added by bastnic 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by laurentj

  • Resolution set to fixed
  • Status changed from new to closed

Bug fixed in the 1.0.x branch and trunk. Thanks.

Changed 12 years ago by bastnic

comment:2 Changed 12 years ago by bastnic

The patch also resolved a problem with the zipFilename which don't have a majuscule F.

And I forgot to change copyright and contributor. Sorry.

comment:3 Changed 12 years ago by bastnic

  • Resolution fixed deleted
  • Status changed from closed to reopened

I just seen that you don't applied my patch, so the problem with the zipFilename isn't resolved. Do I have to open a new ticket ?

comment:4 Changed 12 years ago by laurentj

Sorry, but I fixed the bug before you've submited your patch. However you're right about the "F". I will fix it. Thanks for your reports.

comment:5 Changed 12 years ago by bastnic

this bug is closed, I open a new.

comment:6 Changed 12 years ago by bastnic

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.