似乎新版本的qqconnect插件和掌上論壇插件沖突

kidozh

RT，我發現最近使用掌上論壇插件訪問論壇503，比如這個API：


http://m.9999xn.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

希望看看如何解決這個問題。

湖中沉

用手機版訪問似乎是正常的

kidozh

湖中沉 發表于 2022-12-27 12:19
用手機版訪問似乎是正常的

對 但是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關了？

湖中沉

kidozh 發表于 2022-12-27 13:27
對 但是mobile插件的API返回都有問題，我檢索了以下`tpl_global_login_extra`這個函數，發現在php里面確 ...

嗯，討論感覺不適合做引用傳遞，建議PR直接修改mobile插件相關功能點的代碼

kidozh

湖中沉 發表于 2022-12-27 13:33
嗯，討論感覺不適合做引用傳遞，建議PR直接修改mobile插件相關功能點的代碼 ...

好吧 但是官方怎么手動控制這個插件呢，是要隨著大版本升級嘛？

PS：我剛剛測試了一下，關閉了qqconnect就能正確加載api，最近一直在看推送插件就沒管這個新老版本的兼容性哈哈哈

kidozh

湖中沉 發表于 2022-12-27 13:33
嗯，討論感覺不適合做引用傳遞，建議PR直接修改mobile插件相關功能點的代碼 ...

搞定！就是沒有定義的問題，我直接從module.htm復制eval內容到php里面，這樣就定義了，但是以后這個重復定義的問題也會存在，而且也不是很利于module.htm和那啥的分離。

一會我去發個PR

湖中沉

kidozh 發表于 2022-12-27 13:37
好吧 但是官方怎么手動控制這個插件呢，是要隨著大版本升級嘛？

PS：我剛剛測試了一下，關閉了qqconnect ...

應用中心可以更新插件版本的

kidozh

湖中沉 發表于 2022-12-27 13:46
應用中心可以更新插件版本的

那也行，那我去改mobile插件吧，希望官方能不能剝離這兩塊，及時更新？

湖中沉

kidozh 發表于 2022-12-27 13:56
那也行，那我去改mobile插件吧，希望官方能不能剝離這兩塊，及時更新？

mobile插件是可以單獨更新的，放心

kidozh

湖中沉 發表于 2022-12-27 13:59
mobile插件是可以單獨更新的，放心

好的，那我最近去看一下