LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-12-29 06:02:07 +03:00
Code Issues Packages Projects Releases Wiki Activity

Spreadsheet: Ensure that cell exists on drop event for text data

Browse Source 
Fixes crash where we tried to get a cell to set text from the drop
event. We now create the cell if it does not already exists.

Fixes issue #5923
This commit is contained in:
David Isaksson 2021-03-23 22:08:16 +01:00 committed by Andreas Kling
parent 6179c9ad25
commit 391d1ddfd8
Notes: sideshowbarker 2024-07-18 21:06:47 +09:00 
Author: https://github.com/Granddave 🔰
Commit: https://github.com/SerenityOS/serenity/commit/391d1ddfd87
Pull-request: https://github.com/SerenityOS/serenity/pull/5928
Issue: https://github.com/SerenityOS/serenity/issues/5923
Reviewed-by: https://github.com/alimpfard ✅
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Userland/Applications/Spreadsheet/SpreadsheetView.cpp
View File

@ -297,10 +297,8 @@ SpreadsheetView::SpreadsheetView(Sheet& sheet)
}
if (event.mime_data().has_text()) {
auto* target_cell = m_sheet->at({ (size_t)index.column(), (size_t)index.row() });
VERIFY(target_cell);
target_cell->set_data(event.text());
auto& target_cell = m_sheet->ensure({ (size_t)index.column(), (size_t)index.row() });
target_cell.set_data(event.text());
return;
}
};