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
Change by Kaan Kivilcim - 06/Feb/09 12:38 AM
Field Original Value New Value
Attachment screenshot-1.jpg [ 11646 ]

Change by Kaan Kivilcim - 06/Feb/09 12:38 AM
Attachment screenshot-2.jpg [ 11647 ]

Change by Kaan Kivilcim - 06/Feb/09 12:39 AM
Attachment screenshot-1.jpg [ 11648 ]

Change by Kaan Kivilcim - 06/Feb/09 12:40 AM
Attachment screenshot-2.jpg [ 11649 ]

Michael Todd - 09/Feb/09 11:10 PM
Matteo, can you verify/explain the issue noted in this ticket. Please let me know.

Mike


Change by Michael Todd - 09/Feb/09 11:10 PM
Assignee Chris Nutting [ chris.nutting ] Matteo Beccati [ matteo.beccati ]

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.


Change by Matteo Beccati - 09/Feb/09 11:36 PM
Assignee Matteo Beccati [ matteo.beccati ] Chris Nutting [ chris.nutting ]

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.

Change by Matteo Beccati - 10/Feb/09 07:41 AM
Status Needs Verification [ 10000 ] Verified [ 10001 ]

Change by bernard.lange - 12/Feb/09 05:58 PM
Link This issue depends on OX-4902 [ OX-4902 ]

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.

Change by bernard.lange - 12/Feb/09 06:12 PM
Assignee Chris Nutting [ chris.nutting ] Bernard Lange [ bernard.lange ]

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...




Change by andrzej.swedrzynski - 13/Feb/09 03:32 PM
Fix Version/s OpenX Market Plugin v0.1.3-rc1 [ 10754 ]
Fix Version/s Milestone 28 [ 10717 ]

Change by andrzej.swedrzynski - 13/Feb/09 03:40 PM
Link This issue is depended on by OXPL-101 [ OXPL-101 ]

Change by bernard.lange - 18/Feb/09 09:27 AM
Remaining Estimate 0h [ 0 ]
Time Spent 0.3h [ 1080 ]


Change by bernard.lange - 18/Feb/09 09:29 AM
Status Verified [ 10001 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

Change by bernard.lange - 18/Feb/09 09:31 AM
Time Spent 0.3h [ 1080 ] 0.33h [ 1200 ]

Change by Pawel Dachterski - 30/Mar/09 02:19 PM
Status Resolved [ 5 ] Closed [ 6 ]