87 Star 327 Fork 98

RockYang / NKeditor

 / 详情

不知道是不是我的姿势不对,用不起来呀

已完成
创建于  
2017-10-14 12:26

输入图片说明
输入图片说明

问题1:

$base64 = trim($_POST['base64']); // /qiniu/upload_json.php 31行

$_POST中没有base64的字段数据,我加了个isset判断

// 返回结果
    $json = new \JsonResult();  // /qiniu/upload_json.php 51行

实例化JsonResult需要传2个参数 $code, $message

//JsonResult.php 75行
public function __construct($code = null, $message = null){
        $this->setCode($code);
        $this->setMessage($message);
    }

加了允许为空,然后编辑器终于可以正确的上传了
是我哪里姿势不对吗,为什么要改来改去。。。

问题2

输入图片说明

用的laravel框架,搜索图片直接拼上了我当前url了,可是编辑器程序在别的目录呀
修改了下 php/qiniu/image_search_json.php 21行 "thumbURL" => "{$baseUrl}/../image_grap_json.php?img_url={$value["thumb"]}", 加了个baseUrl对上路径就好了

问题3

输入图片说明
整合到框架内的时候,上传返回成功了,可是页面没反应,要是直接打开编辑器里面的index.html上传,都是ok的

找到原因了

输入图片说明
JDialog 没有找到

errorMsgHandler : function(message, type) {
            // JDialog.msg({type:type, content:message, timer:2000});
        }

注释掉调用编辑器的这段代码就好了,可是index.html 里说 JDialog 不是必须引入的,这里却用了这么一句,着实让人困扰

评论 (1)

mean2015 创建了任务
mean2015 更新了任务
mean2015 更新了任务
mean2015 更新了任务
mean2015 更新了任务

首先感谢你的尝试使用 NKeditor,还帮助测出了这么多的bug,确实有些地方没有注意到。
问题一:
应该是你用的是php7.1以上的版本,而且没有关闭 E_WARNNING 和 E_NOTICE 报错, 我本机上用的是php7.0, php7.1是对参数的类型和数量要严格检查的,而php7.0没有,所以我本地测试的时候是没有发现问题了。后期会做php的版本兼容。

问题二:
这个确实是bug,没有考虑到伪静态的地址转发的问题,使用了相对地址

问题三:
JDialog确实不是必须要引入的,因为这个errorMessageHandler 是由用户传入的参数,只不过我的 demo 中使用了 JDialog, 这个完全可以是用 window.alert() 代替的。不过这个确实我的注释和代码写得不够健壮,errorMessageHandler 应该这样写的:

errorMsgHandler : function(message, type) {
     try {
        JDialog.msg({type:type, content:message, timer:2000});
    } catch(Error) {
        alert(message);
    }

}

再次感谢反馈和对 NKeditor 的支持,这些问题都会在下个版本修正 :smile:

RockYang 关闭了任务

登录 后才可以发表评论

状态
负责人
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
参与者(2)
7775 blackfox 1578914513
JavaScript
1
https://gitee.com/blackfox/NKeditor.git
git@gitee.com:blackfox/NKeditor.git
blackfox
NKeditor
NKeditor

搜索帮助