查看: 49|回复: 9

[讨论] 似乎新版本的qqconnect插件和掌上论坛插件冲突

[复制链接]

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
发表于 2022-12-27 11:44:26 | 显示全部楼层 |阅读模式
RT,我发现最近使用掌上论坛插件访问论坛503,比如这个API:


https://www.dismall.com/api/mobi ... mdisplay&fid=48


似乎是qqconnect里面的tpl_global_login_extra()未定义,这个好像是通病,所有升级Discuz 3.5的都会有这个问题:

比如:https://www.marioforever.net/api/mobile/index.php?version=4&module=forumdisplay&fid=48

希望看看如何解决这个问题。
这个, 论坛, 插件, 掌上, 比如
回复

使用道具 举报

39

主题

630

回帖

0

积分

热心网友

金币
630
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-7-27
发表于 2022-12-27 12:19:49 | 显示全部楼层
用手机版访问似乎是正常的
回复

使用道具 举报

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
 楼主| 发表于 2022-12-27 13:27:07 | 显示全部楼层

对 但是mobile插件的API返回都有问题,我检索了以下`tpl_global_login_extra`这个函数,发现在php里面确实没有定义这个函数,只有htm里面有。

我看了以下唯一的一个可能定义了这个函数的地方(我也不知道为啥在module.htm这个文件里,路径/source/plugin/qqconnect/template/module.htm第一行eval)似乎这个只是调用了$_G,但是我不是很清楚这个是有什么实际作用,如果没有的话,官方是否要注释这个?

应该是因为掌上论坛没有加载这个module.htm致使没有eval这个tpl_global_login_extra函数,引发了此次错误,但是我看了以下qqconnect还hook了其他代码,比如这个module.htm下各类的eval,因此也很有可能引发其他错误。

所以应该是要在访问论坛时eval这个函数,从而修复这个未定义的bug。很奇怪老板也是这个问题,但是老版本没有报这个的BUG。

PS,话说我提交的那个PR是不是官方不准备合并进主线了啊,要是不准备的话,那似乎我把那个PR关了?
回复

使用道具 举报

39

主题

630

回帖

0

积分

热心网友

金币
630
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-7-27
发表于 2022-12-27 13:33:24 | 显示全部楼层

嗯,讨论感觉不适合做引用传递,建议PR直接修改mobile插件相关功能点的代码
回复

使用道具 举报

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
 楼主| 发表于 2022-12-27 13:37:15 | 显示全部楼层

好吧 但是官方怎么手动控制这个插件呢,是要随着大版本升级嘛?

PS:我刚刚测试了一下,关闭了qqconnect就能正确加载api,最近一直在看推送插件就没管这个新老版本的兼容性哈哈哈
回复

使用道具 举报

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
 楼主| 发表于 2022-12-27 13:45:16 | 显示全部楼层

搞定!就是没有定义的问题,我直接从module.htm复制eval内容到php里面,这样就定义了,但是以后这个重复定义的问题也会存在,而且也不是很利于module.htm和那啥的分离。

一会我去发个PR
回复

使用道具 举报

39

主题

630

回帖

0

积分

热心网友

金币
630
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-7-27
发表于 2022-12-27 13:46:37 | 显示全部楼层

应用中心可以更新插件版本的
回复

使用道具 举报

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
 楼主| 发表于 2022-12-27 13:56:37 | 显示全部楼层

那也行,那我去改mobile插件吧,希望官方能不能剥离这两块,及时更新?
回复

使用道具 举报

39

主题

630

回帖

0

积分

热心网友

金币
630
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-7-27
发表于 2022-12-27 13:59:05 | 显示全部楼层

mobile插件是可以单独更新的,放心
回复

使用道具 举报

9

主题

25

回帖

0

积分

热心网友

金币
25
阅读权限
220
精华
0
威望
0
贡献
0
在线时间
0 小时
注册时间
2010-10-12
 楼主| 发表于 2022-12-27 14:02:20 | 显示全部楼层

好的,那我最近去看一下
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

相关侵权、举报、投诉及建议等,请发 E-mail:qiongdian@foxmail.com

Powered by Discuz! X5.0 © 2001-2026 Discuz! Team.

在本版发帖返回顶部