Just found one location where split is used.
Patched it with preg_split since split has more than one delimiter.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz’s picture

Status: Needs review » Reviewed & tested by the community

Split? How quaint.

dawehner’s picture

Status: Reviewed & tested by the community » Fixed

Commited to 7.x together with a split fix in another profile related file.
Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

wanjee’s picture

Hello,

Will the patch also be applied on Views 6.x ? I can find the same split() call in Views 6.x-3.0-alpha3.

Thanks,

Seb

bojanz’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Closed (fixed) » Patch (to be ported)

I see it as well (in 6.x-3.x-dev), let's fix it.

Btw, please don't use alpha3. It's more than a year old. Use -dev.

Avalanche’s picture

FileSize
163.25 KB

I applied this patch, and the editing screen now shows the various displays under each other, instead of being able to use the tab on the left to switch between displays.

I've attached a screenshot.

Here is the command -- with output -- I used to apply patch:

patch -p0 < views-avoid-using-split-since-its-deprecated-by-php5_3.patch 
patching file modules/profile/views_handler_filter_profile_selection.inc
Hunk #1 succeeded at 13 (offset -1 lines).

Thanks,
Ace

dawehner’s picture

This output seems to be fine.

Avalanche’s picture

The dev version you recommended in http://drupal.org/node/1191066 did the trick for me! Thank you so much! :)

dawehner’s picture

Status: Patch (to be ported) » Fixed

And backported.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.