jasonrogena / jasonrogena/php-excel-reader

PHP 5.3.3+ deprecated fix

Open
#162 0 comments 0 reactions 0 assignees View on GitHub
auto-migrated Priority-Medium Type-Defect
Dominant language
PHP
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

```
Attached version for new versions of PHP where passing by refference and
split() is deprecated.

(removed & on 916 + split -> explode)
```

Original issue reported on code.google.com by `viki...@gmail.com` on 18 Jan 2013 at 3:44

Attachments:
- [excel_reader2new.php](https://storage.googleapis.com/google-code-attachments/php-excel-reader/issue-162/comment-0/excel_reader2new.php)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the attached excel_reader2new.php and the corresponding code in the repository, focusing on the reference marker near line 916 and uses of split(). Confirm the compatibility changes by checking that these deprecated constructs are replaced as described and that the reader still handles its existing Excel input.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
data
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.