History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: OXPL-120
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: bernard.lange
Reporter: Kaan Kivilcim
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OpenX Ad Server Plugins

Cannot complete sign up for OpenX market, dies with PHP fatal error

Created: 06/Feb/09 12:35 AM   Updated: 30/Mar/09 02:19 PM
Component/s: OpenX Market Plugin
Affects Version/s: OpenX Market Plugin v0.1.2-beta
Fix Version/s: Milestone 28, OpenX Market Plugin v0.1.3-rc1
Security Level: Public (All users can see these issues)

Time Tracking:
Original Estimate: Not Specified
Remaining Estimate: 0h
Time Spent - 0.33h
Time Spent: 0.33h
Time Spent - 0.33h

File Attachments: None
Image Attachments:

1. screenshot-1.jpg
(42 kb)

2. screenshot-2.jpg
(23 kb)
Environment: lighttpd 1.4.20, php 5.2.8, mysql 5.1.30 w/ innodb tables, fedora 9
Issue Links:
Depends


 Description  « Hide
After installing the 2.7.29-beta and attempting to sign up for the OpenX Market I receive the following error after trying to associate an existing OpenX account (see images attached).

Fatal error: Call to undefined function ctype_alnum() in /var/www/<REMOVED>/openx-2.7.29-beta/lib/Zend/Uri.php on line 96



 All   Comments   Work Log   Change History   FishEye   Crucible   Builds      Sort Order: Ascending order - Click to sort in descending order
Michael Todd - 09/Feb/09 11:10 PM
Matteo, can you verify/explain the issue noted in this ticket. Please let me know.

Mike


Matteo Beccati - 09/Feb/09 11:36 PM - edited
ctype extension might not be compiled in, preg_match should be used instead.

Edited: it seems it's Zend Framework using ctype_* functions, so a decision has to been made about either patching ZF or requiring the ctype extension on install/upgrade of OpenX.


Kaan Kivilcim - 10/Feb/09 12:11 AM
just to confirm for you guys, the ctype extension is not compiled in php. is this going to be a required extension moving forward or will there be fallback code in the plugin? i'll recompile php to include ctype today so that I can go about testing the plugin, etc.

According to OX-4902 ctype has been made explicitly required in OpenX 2.7.30-beta, we need to bump required version for market plugin as well.

Chris Nutting - 12/Feb/09 06:53 PM
I would hold off on this for now...

We may add ctype compatibility functions from http://upgradephp.berlios.de/ to core - which would negate the need for the ctype check and a version bump in the plugin...