Skip to content

Fix: Fix column ignorance on import - #2877

Open
Koc wants to merge 3 commits into
mainfrom
bugfix/fix-column-ignorance-on-import
Open

Fix: Fix column ignorance on import#2877
Koc wants to merge 3 commits into
mainfrom
bugfix/fix-column-ignorance-on-import

Conversation

@Koc

@Koc Koc commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Reproducer

  1. create table with 2 columns
  2. start import csv file with 3 columns (2 match to existing columns, 1 extra). Unselect "create new columns"
  3. ignore 3rd column
  4. import

Expected result: file imported, ignored column not imported

Actual result: http 500 with an error Call to a member function getId() on string in file '/var/www/html/apps/tables/lib/Service/ImportService.php' line 565

nextcloud-tables-ignorance-2026-08-08_23.04.30.mp4

馃弫 Checklist

  • 鉀戯笍 Tests (unit and/or integration) are included or not needed
  • 馃敊 Backport requests are created or not needed: /backport to stableX.X
  • 馃搮 Milestone is set
  • 馃尭 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@Koc
Koc marked this pull request as ready for review August 8, 2026 21:30
@Koc
Koc requested review from blizzz and enjeck as code owners August 8, 2026 21:30
Koc added 2 commits August 8, 2026 23:51
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc
Koc force-pushed the bugfix/fix-column-ignorance-on-import branch from ab95ef0 to a0a3147 Compare August 8, 2026 21:51
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant