Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix deprecated warnings and adjust function calls #197

Merged
merged 13 commits into from
Nov 16, 2024

Conversation

myml
Copy link
Member

@myml myml commented Nov 16, 2024

解决所有弃用警告,为适配Qt6做准备

myml added 12 commits November 16, 2024 14:28
单参数的QProcess::startDetached已弃用, 使用双参数的函数替换
使用DPaletteHelper和DGuiApplicationHelper替换已标记弃用的DApplicationHelper
使用startOfDay替换已标记弃用的构造函数
DHiDPIHelper已标记弃用
Copy link

  • 敏感词检查失败, 检测到2个文件存在敏感词
详情
{
    "calendar-client/src/main.cpp": [
        {
            "line": "        app->setApplicationAcknowledgementPage(\"https://www.deepin.org/acknowledgments/dde-calendar\");",
            "line_number": 67,
            "rule": "S35",
            "reason": "Url link | f82e0b7f80"
        }
    ],
    "calendar-client/src/widget/calendarmainwindow.cpp": [
        {
            "line": "            url = \"https://www.deepin.org/zh/agreement/privacy/\";",
            "line_number": 906,
            "rule": "S35",
            "reason": "Url link | 20e2eab189"
        },
        {
            "line": "            url = \"https://www.uniontech.com/agreement/privacy-cn\";",
            "line_number": 908,
            "rule": "S35",
            "reason": "Url link | 4850a00dd7"
        },
        {
            "line": "            url = \"https://www.deepin.org/en/agreement/privacy/\";",
            "line_number": 912,
            "rule": "S35",
            "reason": "Url link | 38d42f63bf"
        },
        {
            "line": "            url = \"https://www.uniontech.com/agreement/privacy-en\";",
            "line_number": 914,
            "rule": "S35",
            "reason": "Url link | f82409d3b5"
        }
    ]
}

calendar-client/src/customWidget/scheduleview.cpp Outdated Show resolved Hide resolved
@@ -10,7 +10,7 @@
#include "cscheduleoperation.h"
#include "lunarmanager.h"

#include <DHiDPIHelper>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

@@ -11,7 +11,7 @@

#include <DMessageBox>
#include <DPushButton>
#include <DHiDPIHelper>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

@@ -14,7 +14,7 @@
#include "units.h"
#include "commondef.h"

#include <DHiDPIHelper>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

@@ -11,7 +11,7 @@
#include "constants.h"
#include "scheduledaterangeinfo.h"

#include <DHiDPIHelper>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

@@ -324,7 +324,7 @@ void CAllDayEventWeekView::mouseDoubleClickEvent(QMouseEvent *event)
void CAllDayEventWeekView::wheelEvent(QWheelEvent *event)
{
//若滚轮事件为左右方向则退出
if (event->orientation() == Qt::Orientation::Horizontal) {
if (event->angleDelta().x() != 0 ) {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if (event->angleDelta().x() != 0 ) {
if (event->angleDelta().x() != 0) {

@@ -29,12 +29,11 @@
#include "units.h"
#include "commondef.h"

#include <DHiDPIHelper>

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

calendar-client/src/widget/weekWidget/weekheadview.cpp Outdated Show resolved Hide resolved
解决多个警告的小问题
Copy link

  • 敏感词检查失败, 检测到2个文件存在敏感词
详情
{
    "calendar-client/src/main.cpp": [
        {
            "line": "        app->setApplicationAcknowledgementPage(\"https://www.deepin.org/acknowledgments/dde-calendar\");",
            "line_number": 67,
            "rule": "S35",
            "reason": "Url link | f82e0b7f80"
        }
    ],
    "calendar-client/src/widget/calendarmainwindow.cpp": [
        {
            "line": "            url = \"https://www.deepin.org/zh/agreement/privacy/\";",
            "line_number": 906,
            "rule": "S35",
            "reason": "Url link | 20e2eab189"
        },
        {
            "line": "            url = \"https://www.uniontech.com/agreement/privacy-cn\";",
            "line_number": 908,
            "rule": "S35",
            "reason": "Url link | 4850a00dd7"
        },
        {
            "line": "            url = \"https://www.deepin.org/en/agreement/privacy/\";",
            "line_number": 912,
            "rule": "S35",
            "reason": "Url link | 38d42f63bf"
        },
        {
            "line": "            url = \"https://www.uniontech.com/agreement/privacy-en\";",
            "line_number": 914,
            "rule": "S35",
            "reason": "Url link | f82409d3b5"
        }
    ]
}

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, myml

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@myml myml merged commit 8d11e3b into linuxdeepin:master Nov 16, 2024
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants