FineUI 官方论坛

 找回密码
 立即注册

QQ登录

只需一步,快速开始

本论坛已关闭(禁止注册、发帖和回复)
请移步 三石和他的朋友们

FineUI首页 WebForms - MVC & Core - JavaScript 常见问题 - QQ群 - 十周年征文活动

FineUI(开源版) 下载源代码 - 下载空项目 - 获取ExtJS - 文档 在线示例 - 版本更新 - 捐赠作者 - 教程

升级到 ASP.NET Core 3.1,快、快、快! 全新ASP.NET Core,比WebForms还简单! 欢迎加入【三石和他的朋友们】(基础版下载)

搜索
查看: 5810|回复: 7
打印 上一主题 下一主题

【已确认】FineUi开源版,表格删除行的BUG

[复制链接]
跳转到指定楼层
楼主
发表于 2016-2-3 14:46:42 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
反馈BUG
程序版本: 4.2.2和4.2.3
浏览器: Chrome IE8 
BUG截图:
BUG地址: http://fineui.com/demo/#/demo/grid/grid_editor_cell_newdelete.aspx
demo地址:http://fineui.com/demo/#/demo/grid/grid_editor_cell_newdelete.aspx
也有这个BUG,是JS问题
通过demo页面上的“新增数据”按钮给表格新增数据,然后点击表格最右侧的删除列,删除新增行的其中一行,但是每次点击,不管点击删除哪一行新增行,表格删除的都是第一行新增行。跟踪了一下,发现“ext-part2.js‘中的获取选中的行的方法f_getSelectedRows有点问题,record.getId()得到的是undefine,导致f_deleteSelectedRows方法在执行时默认删除了第一行新增行。估计是每一行新增行的id都是undefine,所以删除时只删除第一行,不知道能不能修正呢?
// 获取选中的行
        f_getSelectedRows: function () {
            var me = this, selectedRows = [];
            var sm = me.getSelectionModel();
            if (sm.getSelection) {
                var selection = sm.getSelection();
                var store = me.getStore();
                Ext.Array.each(selection, function (record, index) {
                   selectedRows.push(record.getId());
                });
            }
            return selectedRows;
        },

本帖子中包含更多资源

您需要 登录 才可以下载或查看,没有帐号?立即注册

x
沙发
发表于 2016-2-4 10:46:47 | 只看该作者
你没保存数据
板凳
 楼主| 发表于 2016-2-14 08:37:17 | 只看该作者

前两个版本是没这个问题的,用户操作上不一定要保存再删除行的,可能添加了几行,突然不想要其中一行,然后删除了再保存,但是删除就出现问题了。
地板
 楼主| 发表于 2016-2-16 09:33:53 | 只看该作者
没人回复一吗?麻烦修复一下这个BUG
6#
发表于 2016-2-24 10:30:21 | 只看该作者
我也是遇到这个问题,老是检查自己代码,原来是这样,郁闷。
7#
发表于 2016-2-29 14:33:12 | 只看该作者
的确有这样的问题,还没找到解决办法
8#
发表于 2016-2-29 18:38:48 | 只看该作者
多谢指出,下个版本会修正(v4.2.4)。

由于无法取到新增行的 id,所以我采取了变通的方法,通过 rowIndex 来删除当前选中行。

  1. -修正删除当前选中的新增行时总是删除第一行的问题(lrjnew-8004)。
复制代码
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|FineUI 官方论坛 ( 皖ICP备2021006167号-1 )

GMT+8, 2024-5-11 17:14 , Processed in 0.052729 second(s), 20 queries , Gzip On.

Powered by Discuz! X3.4

© 2001-2017 Comsenz Inc.

快速回复 返回顶部 返回列表