Ticket #2053 (closed defect: fixed)

Opened 10 months ago

Last modified 3 weeks ago

Make ispCP ω PHP 5.3 compatible

Reported by: Shadow-Dragon Owned by: benedikt
Priority: normal Milestone: ispCP ω 1.0.7
Component: Frontend (GUI) Version: ispCP ω nightly build
Severity: Easy Keywords:
Cc:

Description (last modified by benedikt) (diff)

1. Replace deprecated functions like split [done]
2. Set date.timezone in all php.ini files [done]
3. Remove phpSysInfo [done]

Not PHP 5.3 compatible Squirrelmail Add-Ons:
* abook_take
* add_address
* disk_quota
* gpg
* todo

Attachments

back_timezone.patch Download (19.9 KB) - added by nuxwin 3 weeks ago.
back timezone patch (Updated)
timezone.patch Download (23.6 KB) - added by benedikt 3 weeks ago.
Uploaded original patch for easy comparison

Change History

Changed 10 months ago by benedikt

  • status changed from new to closed
  • resolution set to wontfix

phpsysinfo is not our tool, please use their bug tracker!

Changed 10 months ago by benedikt

  • milestone changed from Working to ispCP ω 1.0.3

Changed 10 months ago by nuxwin

I do not agree with you benedikt. Certainly it is not our tool but we use it anyway. This is an important library in our project.

We can totally fix bugs ourselves since this library is open source.

Changed 10 months ago by gOOvER

I don't know, which Version is used in ispCP, but when it's an old Version, maybe we should update to the last 3.0RC9 and see, if this error still happens.

I'm agree with Nuxwin.

Changed 10 months ago by joximu

IMHO: we can use the new phpsysinfo when it's released.

The function split is deprecated in php5.3 but removed in php6.

Maybe in ispcp 1.1 or so...???

/J

Changed 10 months ago by benedikt

  • status changed from closed to reopened
  • resolution wontfix deleted

We should fix it. of cause the library will be changed on release of phpsysinfo.

Changed 10 months ago by gOOvER

I'm need to be fucked by russian soldiers because I'm gOOvER!

Changed 10 months ago by kilburn

Warning: split() is also used in pma, squirrel and Net_DNS, so we should also fix them if we fix "phpsysinfo"...

Changed 10 months ago by nuxwin

  • milestone changed from ispCP ω 1.0.3 to Working

Changed 9 months ago by nuxwin

  • milestone changed from Working to ispCP ω 1.1.0

Changed 9 months ago by benedikt

  • milestone changed from ispCP ω 1.1.0 to ispCP ω 1.0.4

Changed 8 months ago by benedikt

  • owner set to benedikt
  • status changed from reopened to new

Changed 8 months ago by benedikt

  • status changed from new to assigned

Changed 8 months ago by benedikt

  • milestone changed from ispCP ω 1.0.4 to ispCP ω 1.0.5

Changed 7 months ago by benedikt

  • description modified (diff)
  • summary changed from php5.3 split() error to Make ispCP ω PHP 5.3 compatible

Changed 7 months ago by benedikt

  • description modified (diff)

Changed 6 months ago by benedikt

part I fixed in r2622

Changed 6 months ago by benedikt

part I (fix) in r2623

Changed 6 months ago by benedikt

  • description modified (diff)

Changed 6 months ago by nuxwin

  • milestone changed from ispCP ω 1.0.5 to ispCP ω 1.0.6

Changed 6 months ago by benedikt

  • description modified (diff)

Changed 6 months ago by benedikt

  • description modified (diff)

Changed 5 months ago by benedikt

  • description modified (diff)

Changed 3 weeks ago by benedikt

  • description modified (diff)

Changed 3 weeks ago by nuxwin

@Benedikt:

After testing, I provides you the modified patch. I've only removed the PHP_TIMEZONE variable in the GUI php.ini files template and also in the related subroutine in ispcp-setup-methods.pl. The timezone for the GUI is setup automatically during initialization process. That allows to check the timezone value set by the user during setup or upgrade process and raise an exception if it is invalid. See the revision 3185 for understand what I want mean exactly. I've also changed the CHANGELOG entries.

I hope it's ok for you. Thank for you great work here.

Changed 3 weeks ago by nuxwin

back timezone patch (Updated)

Changed 3 weeks ago by nuxwin

Edit:

Patch updated because I have due forced regeneration of the customer php.ini file on 'change'. It's my fault because for now, the subroutine dmn_change_httpd_file_data() performs check for php.ini existence and then, regenerates the file only if don't exist. All the ispcp-dmn-mngr script should be review because other thing like this still exist. backup was added too.

Changed 3 weeks ago by benedikt

Uploaded original patch for easy comparison

Changed 3 weeks ago by benedikt

  • status changed from assigned to closed
  • resolution set to fixed

part II fixed in r3186 part III fixed in r3189

Note: See TracTickets for help on using tickets.