Phpspreadsheet: Remove all deprecated things

Created on 6 Oct 2016  路  5Comments  路  Source: PHPOffice/PhpSpreadsheet

Deprecated things should either be dropped entirely and their replacement be documented, or the @deprecated mentions should be dropped. As noted in https://github.com/PHPOffice/PHPExcel/issues/1029 it's unfortunately not always clear why it was deprecated in the first place.

enhancement

Most helpful comment

This issue is specifically for already deprecated things that need to be dropped before the first release. We are beyond the deprecating phase (by several years in some cases). While I agree with you for future deprecations, I don't think it applies here.

All 5 comments

This might be a clear way of deprecation:

  1. mark the method or code block as @deprecated and document a replacement
  2. after x _time-unit_ (e.g. 1/2 year?) remove the deprecated code

This issue is specifically for already deprecated things that need to be dropped before the first release. We are beyond the deprecating phase (by several years in some cases). While I agree with you for future deprecations, I don't think it applies here.

@MarkBaker I worked on removing deprecated methods and documenting their replacements.

But I got stuck on getCalculatedValue(), setFormulaAttributes() and setFormulaAttributes(), because I couldn't find any replacement. They've been deprecated in 59932b0cac4aec4ba13b17f9f36b0b939bc083c4 and the changelog mention that replacements will be added in the future, but it seems that it's never been done. It seems that everything is related to https://github.com/PHPOffice/PHPExcel/issues/9. Is it still something that you are working on ? What should we do about getCalculatedValue() ?

getCalculatedValue() seems to be intensively used by PHPSpreadsheet: https://github.com/PHPOffice/PhpSpreadsheet/search?q=getCalculatedValue&type=Code&utf8=%E2%9C%93

Following the lack of input from @MarkBaker I took the iniative to un-deprecate those last few methods.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

garnold picture garnold  路  5Comments

isopen picture isopen  路  3Comments

cheinle picture cheinle  路  3Comments

Blacknife picture Blacknife  路  4Comments

ariefbayu picture ariefbayu  路  3Comments