Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
Mark Baker
@MarkBaker
Yes. I did, but it's resolvable when I get home... either reverting the lock file, or enforcing doctrine/instantiator 1.0.5 in the composer.json
Adrien Crivelli
@PowerKiKi
:+1:
Mark Baker
@MarkBaker
But checking on that highlights the same issue/resolution for my calc engine matrix branch... And fixing there will allow me to continue rewriting the trend lines code
I do like that new github feature for offering/accepting changes when reviewing code.... it's pretty cool
Adrien Crivelli
@PowerKiKi
yup, never could accept it yet, but suggesting it is nice
btw, I noticed that your libs use slightly different convention for folder names: classes, unitTests VS src, tests. I don't mind all that much, but I thought if you don't have any specific reason it might be nice to be coherent with PhpOffice projects
Mehdi Chaouch
@mehdichaouch
Hello
Mehdi Chaouch
@mehdichaouch
Someone know why SWITCH function is not supported (my supposition is based on lib/Phpspreadsheet/vendor/phpoffice/phpspreadsheet/src/PhpSpreadsheet/Writer/Xls/Parser.php:215)
and do you have any alternative better that IF imbrication... like this : =IF(plop,1,IF(flip,2,IF(clac,3,0))))
Mehdi Chaouch
@mehdichaouch
(Nested IFs)
Mark Baker
@MarkBaker
Probably it doesn't work because MS Excel never had a switch() function supported by BIFF-format xls files
It's only been introduced very recently, and is only supported for Office 365, not even for Excel 2016
So PHPSpreadsheet doesn't provide any support for it yet
Mehdi Chaouch
@mehdichaouch
@MarkBaker it's from 2016 of course
do you know if implementing this is hard work ?
Mark Baker
@MarkBaker
"Note: This feature is not available in Excel 2016 unless you have an Office 365 subscription. If you are an Office 365 subscriber, make sure you have the latest version of Office."
SWITCH() isn't necessarily hard to implement, just time-consuming; but it wouldn't ever be available for the Xls Writer
Mehdi Chaouch
@mehdichaouch
ok.
Thank you for your feedback
I'll try to make a function to work with Nested IFs
to be sure to have cross compatibility
Marcus Uerlings
@muerlings_twitter
Hi

I´m having a probem with HTML writer when adding nuber format like:
'- * #,##0.00-;[Red]- #,##0.00-;- "-"??-;-@_-'

It displays with html tag span and style on browser

Marcus Uerlings
@muerlings_twitter
instead of just displaying the number in red.
Does someone had the same problem?
Sean Pearce
@smpita
PHPSpreadsheet edited spreadsheets are breaking CenturyLink's efax service resulting in an empty cells if it's formatted as currency. If the document is opened in Excel and immediately saved, the spreadsheet faxes normally. This issue occurs regardless of the type of change made by PHPSpreadsheet -- currently we're just resizing columns from a downloaded report. The unedited report faxes normally. Any ideas?
Fräntz Miccoli
@frantzmiccoli

@MarkBaker thanks for your reply on PHPOffice/PhpSpreadsheet#788 do you think that it is feasible to "hack" the stack based parser in order to give a specific processing to IF calls ?

I am deep in processTokenStack(), still trying to figure what's happening around. If you think it is close to impossible to implement something without generating side effects I will stop there.

Maria Fernanda Orozco Dominguez
@mariaorozco18_gitlab
Buenos dias
alguien por aqui?
Tengo una duda con la libreria y me gustaria saber si me pueden colaborar
Mark Kevin Besinga
@besingamkb
hi. I have a cell that has a float or a double type value. how can I get the cell but in the string format?
Mark Baker
@MarkBaker
Well PHPSpreadsheet has a getFormattedValue() method if the cell has a number format mask... or PHP has functions like number_format()... or even just cast the value to a string
Diego Drigani
@drigani
Hi All! I've a problem with the calculated value on an array formula {=+MIN(IF(AF4:BN4>0;AF4:BN4;""))}
are they supported?
Adrien Crivelli
@PowerKiKi
@drigani no, they are not supported
dirtyluke
@dirtyluke
it seems that PhpSpreadsheet doesn't get chart in when its description wrapped by a oneCellAnchor tag in the drawing.xml file (xlsx type file)
Paul Porzky
@papoms

Hi there,

given an existing xlsx file with lots of formulas and formatting
i want to add some data and write it to a new file.
currently it breaks Formulas and Formatting in the output, even without changes.

$spreadsheet = \PhpOffice\PhpSpreadsheet\IOFactory::load('complextemplate.xlsx');
$writer = \PhpOffice\PhpSpreadsheet\IOFactory::createWriter($spreadsheet, 'Xlsx');
$writer->save("output.xlsx");

Is there a more unobstrusive way of adding data to an existing XLSX File? (Also open for suggestions of other tools / libraries)

Just reading and writing the file results in a corrupted result. Just wondering whether or not i am missing an option or something from the docs to make changes to a file without recreating it complete within PhpSpreadsheet.

Fräntz Miccoli
@frantzmiccoli
I don't know if you only check PR where CI failed, but here PHPOffice/PhpSpreadsheet#816 I don't understand why phpcs is seeing two blank lines when there is only one.
Fräntz Miccoli
@frantzmiccoli
  • where CI didn't fail
Adrien Crivelli
@PowerKiKi
@papoms there is no option to "only add something". It's always a complete rewrite of the entire file. If something changes, then it means it's not supported by the reader and/or the writer. The best course of action is to find out what is missing for your need and contribute the feature. Or find another tool, but I can't recommend any
Fräntz Miccoli
@frantzmiccoli

Hi, I am observing that if A2 contains =HLOOKUP(A1, D1:AC120, 3, FALSE), it works fine if unused lines in the range are using A2 (which otherwise would be a circular reference). Which is consistent with the behavior of spreadsheet software.

But I don't get in the code which part of the calculation engine is handling that and I am observing a bug related to this. One of the "would-be" circular reference is resolved to some value, used in another result that is then cached and retrieved later ... without a proper recomputation.

Mark Baker
@MarkBaker
@PowerKiKi WIth PHP 7.0 officially already at end of life, and 5.6 due to end security-patch support in just 2 days, do you have any strong opinions on whether we should put the minimum PHP version at 7.1 for the next major release? or still provide 7.0 compatibility? With working on the new calc engine, I'm debating whether to use nullable return typehints, which requires 7.1
I'm also going to split off a new developmentVersion2 branch for working on updating for the newer features of PHP7
Mark Baker
@MarkBaker
For the moment I'll leave the composer.json at ^7.0 and set up travis for 7.0, 7.1, 7.2, 7.3
Mark Baker
@MarkBaker
And I'll run a branch off that for my work on the new calc engine.... changes to the actual functions are relatively minor; though the largest is throwing an exception for "Excel" errors (invalid arguments, etc) rather than returning a string containing the error (#VALUE, #NA!, etc); though the function code can be improved as a result of that change
Adrien Crivelli
@PowerKiKi

@MarkBaker we should definitely go with PHP 7.1, otherwise type hinting cannot be as useful as it should be.

For branching I was considering keeping only master branch as a permanent branch. I noticed that develop is not really useful because it should be as stable as possible anyway. So we could just as well merge into master directly (and release from time to time as usual). This simplify our workflow, avoid confusion for new contributors and avoid accidental branch diverging as it happened when releasing 1.5.2.

Adrien Crivelli
@PowerKiKi
I'll take care of removing the develop branch, if that is ok with you ?
Mark Baker
@MarkBaker
👍
Adrien Crivelli
@PowerKiKi
just released 1.6.0
Fräntz Miccoli
@frantzmiccoli

Hi @PowerKiKi

I saw you closed PHPOffice/PhpSpreadsheet#818 this won't be merged ?

jbtsp
@jbtsp
I have seen no info from anyone about this issue PHPOffice/PhpSpreadsheet#770