jasonrogena / jasonrogena/php-excel-reader

developers deprecated? :/

Open
#197 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

```
ISSUES IN LATEST DOWNLOADABLE FILE

01>

Deprecated: Function split() is deprecated in
D:\xampp\htdocs\php-excel-reader-2.21\excel_reader2.php on line 844

FIX : Find-Replace split() to explode()

02>

Deprecated: Assigning the return value of new by reference is deprecated in
D:\xampp\htdocs\jhalmuri\php-excel-reader-2.21\excel_reader2.php on line 916

FIX : Remove & in line 916

$this->_ole =& new OLERead();

should be

$this->_ole = new OLERead();

/*
** Dear Coder's, do make it as easy as possible for end user
** whatever product you make. Thank You
*/

That's iT ! Now restructure to meet your need :)
```

Original issue reported on code.google.com by `formally...@gmail.com` on 21 Oct 2014 at 12:14

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

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect the attached excel_reader2.php, focusing on lines 844 and 916. Replace the deprecated split() usage with explode() and remove the reference marker from the OLERead instantiation, then verify that the reported deprecation messages no longer appear when the reader is used.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.