Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

2.1.5版本更新 #102

Merged
merged 8 commits into from
Apr 3, 2021
Merged

2.1.5版本更新 #102

merged 8 commits into from
Apr 3, 2021

Conversation

yuantuo666
Copy link
Owner

准备更新

@yuantuo666
Merge pull request #101 from yuantuo666/master
1ffa346 
同步
@yuantuo666
增加账号状态显示页面；删除不必要的版本检查，仅保留首页的版本检查；
7342991 
function.php检测账号失效状态；将index.php中获取BDUSS移动到function.php中；
将版权链接修改成GitHub仓库
@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

你换电脑了？最近的提交咋都没 GPG 验证？

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

那版本检查是为了防止用户意外操作导致的不完全更新而设置的，目的是使每个文件的版本统一，防止出现问题。

@lc6464 lc6464 self-requested a review April 3, 2021 02:26
@yuantuo666
Copy link
Owner Author

没用原来的那个电脑，这个没搞gpg

感觉这样判断没必要，有时候更新忘了改也会乱报错

@yuantuo666
Copy link
Owner Author

一般都是覆盖安装或者直接pull更新，不会出现问题的

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

感觉这样判断没必要，有时候更新忘了改也会乱报错

其实就是怕用户更新的时候少复制了什么文件，要不然也不用这么麻烦。

@yuantuo666
Copy link
Owner Author

现在已经将config.php文件改成了实时生成，只要覆盖安装就不会出现问题

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

一般都是覆盖安装或者直接pull更新，不会出现问题的

服务器上生产环境直接 Pull 可不太行，一般都是自己覆盖安装的，但是万一少复制了什么文件就麻烦了，或者说某个（或某几个）文件遇到了什么奇怪的冲突，也很麻烦。

API 和 Index 这类直接访问的无论如何都一定要版本检测，其他相关文件你自己取舍吧。

@yuantuo666
Copy link
Owner Author

另外还要完善一下使用方法，README.md里写的太简单了

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

还有 config.php 也必须要注意，这个文件太重要了。

@yuantuo666
Copy link
Owner Author

一般都是覆盖安装或者直接pull更新，不会出现问题的

服务器上生产环境直接 Pull 可不太行，一般都是自己覆盖安装的，但是万一少复制了什么文件就麻烦了，或者说某个（或某几个）文件遇到了什么奇怪的冲突，也很麻烦。

API 和 Index 这类直接访问的无论如何都一定要版本检测，其他相关文件你自己取舍吧。

行，一般都是直接访问index.php的

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

我想着要不要搞一个在线更新？

@yuantuo666
Copy link
Owner Author

我想着要不要搞一个在线更新？

我也在想，这样就可以避免很多问题

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

行，一般都是直接访问index.php的

这里的直接访问包括用 JavaScript 通过 Ajax 访问 API，所以 api.php 也应该要。

@lc6464
Copy link
Collaborator

lc6464 commented Apr 3, 2021

要不QQ上说吧，邮件太多了吧……

@yuantuo666
Copy link
Owner Author

行，一般都是直接访问index.php的

这里的直接访问包括用 JavaScript 通过 Ajax 访问 API，所以 api.php 也应该要。

实际上访问index.php的时候就已经拦截并转跳到安装页面了，api.php 这个倒不一定。

lc6464
lc6464 reviewed Apr 3, 2021
View reviewed changes
Copy link
Collaborator

@lc6464 lc6464 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议都在相关的行里面了。

api.php
@@ -10,7 +10,6 @@
* @link https://space.bilibili.com/88197958
*
*/
$programVersion_API = '2.1.4';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API 还是要有版本检测比较好。

functions.php
}
break;
case 0:
//模式0：使用本地解析
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这太不专业了吧，随便传个东西进去，只要没被 case 到就是本地解析，容易出问题。

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个函数只有输出啊，，

functions.php
* @author Shelley Shyan
* @copyright http://phparch.cn (Professional PHP Architecture)
*/
function time2Units(int $time)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

一个月按30天算，一年按365天算是否会出问题？
Microsoft .NET 的 TimeSpan 都不支持 Day 和 Year.

因为它表示不引用特定开始点或结束点的常规间隔，所以不能用年份和月份来表示，这两种间隔的天数都是可变的。
https://docs.microsoft.com/zh-cn/dotnet/api/system.timespan

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

啊这 能解决吗
我觉得这样没多大问题

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

啊这 能解决吗
我觉得这样没多大问题

理论上来说可以解决，但是很麻烦，毕竟日历过于复杂，微软的计算器中不知用什么办法解决的，但肯定不是一般的方法，可能写了很多 if（猜的）。

settings.php
@@ -10,7 +10,6 @@
* @link https://space.bilibili.com/88197958
*
*/
$programVersion_Settings = '2.1.4';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

settings 这种用户直接访问的最好还是要验证。

@yuantuo666 yuantuo666 linked an issue Apr 3, 2021 that may be closed by this pull request
IDM下载降速 #103
Closed
yuantuo666
yuantuo666 commented Apr 3, 2021
View reviewed changes
Copy link
Owner Author

@yuantuo666 yuantuo666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

测试安装没问题

@yuantuo666 yuantuo666 merged commit ac79402 into master Apr 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lc6464 lc6464 lc6464 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

IDM下载降速
2 participants
@yuantuo666 @lc6464