Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function split() deprecated message in 2.3.2Beta1 #5

Closed
andrewminton opened this issue Jan 22, 2013 · 3 comments
Closed

Function split() deprecated message in 2.3.2Beta1 #5

andrewminton opened this issue Jan 22, 2013 · 3 comments

Comments

@andrewminton
Copy link

Symphony Deprecated Warning: Function split() is deprecated

An error occurred in C:\wamp\www\website\extensions\multilingual_tag_field\fields\field.multilingual_tag.php around line 230

225 $result = array();
226 $field_data = $data;
227
228 $max_lang_tags = 0;
229 foreach( $field_data as $value ){
230 $a = split(',', $value);
231 $max_lang_tags = (count($a) > $max_lang_tags ? count($a) : $max_lang_tags);
232 }
233
234 foreach( FLang::getLangs() as $lc ){

Is there a work around for this?

@andrewminton
Copy link
Author

I replaced split() with explode() and it worked. Is this legacy code or have I changed something that could have a knock on effect for other functions? it's in the processRawFieldData function so shouldn't affect anything further down the line no?

@andrewminton
Copy link
Author

I would send a pull request but the change of code is so trivial it's pointless for me to do so. Unless you wanted me to that is :) ?

@6ui11em
Copy link
Collaborator

6ui11em commented Jan 22, 2013

Thanks you, i will try to attend the pull requests and issues soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants