Xournal++插件添加笔画后撤销/重做功能失效问题分析

在Xournal++ 1.2.4开发版本中，用户发现通过插件API（app.addStrokes）添加的笔画无法被撤销操作撤销。经过技术团队深入调查，发现这是一个由代码重构引入的回归问题。

问题现象

当用户通过插件接口添加笔画后：

1. 使用撤销功能时，新添加的笔画不会被撤销
2. 在尝试修复后，重做操作可能导致应用崩溃

技术背景

Xournal++的撤销/重做系统依赖于对文档修改操作的完整记录。当通过插件API添加笔画时，系统需要：

1. 正确创建笔画对象
2. 将笔画添加到文档结构
3. 记录操作到撤销栈

问题根源

通过代码审查和git bisect工具定位，发现问题出现在#5350重构提交的第二个commit（399b10537b596998318f361bf0d610bfc67161fd）。重构过程中遗漏了关键的一行代码：

strokes.push_back(stroke.get());

这行代码原本负责将新创建的笔画对象添加到笔画集合中，以便撤销系统能够跟踪这个操作。

解决方案

修复方案是在添加笔画到文档前，先将其加入笔画集合：

// 修复代码示例
strokes.push_back(stroke.get());  // 将笔画加入集合
addStrokeHelper(L, std::move(stroke));  // 添加笔画到文档

相关技术点

1. 撤销/重做系统实现原理：

   • 使用命令模式记录操作
   • 每个修改操作都会创建对应的撤销动作
   • 动作对象保存足够信息来撤销/重做操作

2. 插件系统交互：

   • 插件通过Lua API与主程序交互
   • 需要确保API调用与内部状态同步

3. 内存管理：

   • 使用智能指针管理笔画对象生命周期
   • 需要注意对象所有权转移

经验总结

1. 重构时需特别注意状态一致性
2. 插件API边界处的测试覆盖很重要
3. 撤销/重做等核心功能需要专项测试

该修复已合并到主分支，将在下一个版本中发布。开发者在使用插件API时应注意检查核心功能是否正常工作。