hoogi91 / spreadsheets Goto Github PK

Description
Drop support for TYPO3 versions < 9.5

Version: 2.0.2Version(s) affected: x.y.z

If an Excel spreadsheets is sorted by date and the date in the format e.g dd.mm.yyyy it will changed in typo 3 10 to the following format: mm.dd.yyyy. This is not desired if it is done on a German Homepage.

Regards

Wallma

How to reproduce

Possible Solution

Additional context

Version(s) affected:
spreadsheets 2.0.1
charts 1.1.0

Description
When editing a charts record in the backend, the spreadsheets DataInputElement is not completey initialized.
The following elements are empty:
<ul class="nav nav-tabs spreadsheet-sheets" style="display:none;"></ul>
<div class="spreadsheet-table" draggable="false" style="display: none"></div>
This only happens on Chrome, though.

How to reproduce
1.) Install TYPO3 10.4
2.) Install spreadsheets 2.0.1
3.) Install charts 1.1.0
4.) Upload an Excel file
5.) Create a charts record and edit it with Chrome.

It sporadically works, which points to a JS timing problem.

Possible Solution
None so far. I'm still looking into it.

Additional context
Screeshots from the backend and the inspector

Description
Could you add the possibility to show all or multiple selected worksheets at once?

Example
I'd like to achieve an excel like view with tabs to switch between the sheets. This ain't possible at the moment (except one uses multiple content elements), right?

Hi hoogie91, vte-udg asked me to contact you about an update problem.

We have updated spreadsheets from 1.1.2 to 2.0.2 in a TYPO3 10.4.17. We are now experiencing the following problem:

Version(s) affected:
2.0.2

Description
ArgumentCountError Too few arguments to function Hoogi91\Spreadsheets\DataProcessing\SpreadsheetProcessor::__construct(), 0 passed and exactly 3 expected

How to reproduce
Call a FE-Page with a spreadsheed content element

Possible Solution
This is one of the best problems: everything should be correct, but it still doesn't work.
The __construct() is correct, the Services.yaml is correct.
I have no idea why it's banging here.

If I replace the __construct() with the following, everything is fine:

public function __construct() {
    /** @var ExtractorService $extractorServicel */
    $this->extractorService = GeneralUtility::makeInstance(ExtractorService::class);

    /** @var StyleService $styleService */
    $this->styleService = GeneralUtility::makeInstance(StyleService::class);

    /** @var PageRenderer $pageRenderer */
    $this->pageRenderer = GeneralUtility::makeInstance(PageRenderer::class);
}

Btw, in "charts", the __construct() is done in the same way as my replacement, charts are working fine.

Maybe you can look at this again and find a solution for this.

Best of wishes

mcsikos

Version(s) affected: Charts 1.0.5 | TYPO3 8.7.29

Description
Tables are not scrollable and the checkboxes are not displayed

How to reproduce

1. Create new diagram in list view
2. Use Excel file with more than 10 rows

Possible Solution

Additional context

I only tested with TYPO3 8.
Extension is awesome otherwise 🎉

Version(s) affected:
2.0.2

Description
Hi hoogie91, we are having problems displaying merged rows here.

Attached are also some screenshots to make it clearer.

When rows are merged, a rowspan is rendered in the fluid. When this is the case, the table is displayed incorrectly in the front-end.

In the old version of the spreadsheets (1.0.2), however, no rowspan is output with the same Excel and the display was correct.

Do you know what the reason could be?

Additional context

Version(s) affected: 3.3.0

Description
When I try to insert the extension element spreadsheets_table, I get an error:
#1438614542 UnexpectedValueException
No or invalid showitem definition in TCA table tt_content for type spreadsheets_table

If I insert the spreadsheets_tabs element, there is no error, and everything works as it should

Additional context
TYPO3 11.5.12
PHP 8.0

First of all a big thank you for this nice extension!

Version(s) affected: 3.3.0, TYPO3 10.4.30

Description
Showing a XSLX file generates lots (more than 1000) empty columns to the right. I tried to set up the print area, but this seems to have no effect.

How can I narrow the generated table to a defined area in the excel sheet?
Or the other way round: How does the extension know, which area of a sheet to show?

Hi hoogie91, vte-udg asked me to contact you about an update problem.

We have updated spreadsheets from 1.1.2 to 2.0.2 in a TYPO3 10.4.17. We are now experiencing the following problem:

Version(s) affected:
2.0.2

Description
Return value of Hoogi91\Spreadsheets\Domain\ValueObject\DsnValueObject::getSheetIndex() must be of the type int, null returned

How to reproduce
Call a BE-Page with a spreadsheed content element

Possible Solution
in legacyDSNParsing() is a condition
if (empty($fullSelection) === false) {...}

Here, $fullSelection is " '0' (1 chars) ", so the "empty($fullSelection)" will be "true".
As a result, it cannot be executed:
"
$this->sheetIndex = (int)($sheetIndex ?: 0);
$this->selection = $selection ?: null;
$this->directionOfSelection = $directionOfSelection ?: null;
"
which leads to a " $this->sheetIndex" is "null"

Maybe you can fix this?

Best of wishes

mcsikos

Hi, I've got a question.

After you have uploaded an Excel file, you have to select the table via the spreadsheetInput. Is it possible somehow to automatically select the first table? Or if there is only one table, it is this one.

I have an import (external_import) of product data with Excel tables, which should be output via this great extension. It would be useful if the first table were selected immediately during saving.

Stephan

Description
Integrate https://github.com/marketplace/actions/codecov into CI and PR workflow.

Example
https://github.com/marketplace/actions/codecov
https://codecov.io/

Version(s) affected: 11.5.29

Description
After installing the extension typo3 backend will be unusable and frontend delivers an empty page.
But installing in an typo3 12 (12.4.3) instance works properly.

How to reproduce
Install it on typo3 11 with Bootstrap extension and you will see.

Possible Solution
Do not know.

Additional context
Hope my infos are enough to reproduce the issue.

Description

With large tables especially on mobile devices it is suitable to configure the first row/first column as fix (no scrolling). Any chance to implement?
It would be perfect, if you can define the number of columns/rows to be fixed.
Example

Description
Improve existing documentation to current state and add more examples and screenshots.
Use existing guide to implement: https://docs.typo3.org/m/typo3/docs-how-to-document/master/en-us/WritingDocForExtension/CreateFromScratch.html

Example
https://docs.typo3.org/p/buepro/typo3-flux-elements/1.0/en-us/
https://docs.typo3.org/p/sebkln/content-slug/1.1/en-us/

Version(s) affected:
2.0.2

Description
Hi hoogie91, we have noticed an extremely strange effect here.

Attached are also a couple of screenshots to make it clearer.

When we embed a spreadsheet, it has no style information, so no colours in the front-end.

If we embed the identical one again, it will have style information.

It is always the first element that is faulty.

This is also reproducible, e.g. when we swap the elements.

Do you know what the reason could be?

Additional context

Description
New Feature/Option for renderType "spreadsheetInput" to allow backend users to decide whether their spreadsheet selection should be extracted column-based. Currently all selections are interpreted row by row (row-based).

Example
hoogi91/charts#2 describes an issue where it isn't possible to decide how the spreadsheet selection should be interpreted (row- or column-based). In the linked issue the expectation is it to allow the backend user to decide that the "dataset labels" should be column-based interpreted/extracted.

Hi,
thanks for the version 2.1.0

But after updating I get the error

Fatal error: Cannot redeclare Matrix\adjoint() (previously declared in .../typo3conf/ext/spreadsheets/Resources/Private/Composer/vendor/markbaker/matrix/classes/src/Functions/adjoint.php:21) in phar:///.../typo3conf/ext/svconnector_xls/vendors.phar/vendor/composer/../markbaker/matrix/classes/src/functions/adjoint.php on line 20

Version(s) affected: spreadsheets_1.1.1

Description
Somehow the cell selection function isn't working. I don't know why, maybe you can help me there.
I need the function for an important school project.

Maybe it has something to do with this error typo3 shows me:
Core: Exception handler (WEB): Uncaught TYPO3 Exception: Class 'Hoogi91\Spreadsheets\Utility\ExtensionManagementUtility' not found | Error thrown in file C:\xampp\htdocs\typo3\typo3conf\ext\spreadsheets\Configuration\TCA\Overrides\tt_content.php in line 6. Requested URL: http://localhost/typo3/typo3/index.php?route=%%2Ftools%%2FExtensionmanagerExtensionmanager%%2F&token=--AnonymizedToken--

Typo3 cell selection:

Output:

tt_content.php config

Hi,
Is it possible to delete an already occupied field 'spreadsheetInput'? Unfortunately, clicking on the button to the right of the field has no effect.

Thanks
Stephan

Version(s) affected: 1.1.2

Description
In the documentation, it is stated that the default value for sheetsOnly is false (https://docs.typo3.org/p/hoogi91/spreadsheets/master/en-us/Configuration/Index.html). But in the Configuration/TCA/Overrides/tt_content.php, the value is set to true.

How to reproduce

• Install version 1.1.2 of the extension
• Inspect Configuration/TCA/Overrides/tt_content.php, line 81
• The value for sheetsOnly is true

Description
Handsontable is currently loaded as full build library.
Future releases should provide a custom build that is as light as possible in comparison to current full build.

For informations on how to create custom builds see: https://handsontable.com/docs/7.1.0/tutorial-custom-build.html

Hi Thorsten,

I came across your spreadsheets extension through your update and tested it briefly. Really nice!

Very nice is the selection of table cells in the backend.

What I didn't manage to do is to output the headers of a tables. Is there a way to do this?

Then I got the idea, wouldn't it be cool if you could mark and cells, as you can also do with the area?

Alternatively, an option "Table header position" with none/top/left like the content element tables hast, would be nice.

Version(s) affected: 3.3.0, TYPO3 11.5 BE

Description
When I try to add new Table(Excel Spreadsheets) content in the BE I get a warning message.

How to reproduce

1. Go to the TYPO3 11.5 BE
2. Create a new page
3. Add new "Typical page content".
4. Choose Table(Excel Spreadsheets)

Possible Solution
A hotfix for the problem is to add ?? null in the if statement in CellService.php line 43.
It only fixes the warning message.
if ($this->currentLocales === '' && is_int($GLOBALS['TSFE']->id ?? null)) {

Additional context
(1/1) #1476107295 TYPO3\CMS\Core\Error\Exception
PHP Warning: Undefined global variable $TSFE in /var/www/html/htdocs/typo3conf/ext/spreadsheets/Classes/Service/CellService.php line 43

got following error on TER installation:

Oops, an error occurred! Class 'PhpOffice\PhpSpreadsheet\Reader\Xlsx' not found

TYPO3 8.7.22
spreadsheets 1.0.1

Description
Migrate continous integrations from travis to Github workflows. Also integrate TYPO3 extension TER upload in Github's workflow

Example
https://github.com/the-coding-owl/action-typo3-extension-repository-upload