GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — 新功能：小程序的发货信息管理，第三方平台的购物订单管理部分接口，添加获取订单页path信息，申请设置订单页path信息接口 by Molzx · Pull Request #3083 · binarywang/WxJava

Conversation

Molzx commented Jul 17, 2023

Copy link

Copy Markdown

Contributor

新功能：

小程序的发货信息管理服务的全部接口
第三方平台的基础信息的添加获取订单页path信息，申请设置订单页path信息接口
第三方平台的购物订单的上传购物详情，上传物流信息，上传合单购物详情，上传合单物流信息，开通购物订单产品权限

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

xzh added 5 commits

May 23, 2023 14:43


          添加第三方平台购物订单管理相关

7ebb7ca

(cherry picked from commit ebc31ce)


          添加获取订单页path信息，申请设置订单页path信息接口

9ad2746


          小程序发货信息录入，合单信息录入，是否开通功能接口添加

f4ac50c


          补全发货信息相关接口

90c909c


          Merge branch 'up-develop' into develop

4044da0

# Conflicts:
#	weixin-java-miniapp/src/main/java/cn/binarywang/wx/miniapp/api/WxMaService.java
#	weixin-java-miniapp/src/main/java/cn/binarywang/wx/miniapp/api/impl/BaseWxMaServiceImpl.java

sonatype-lift Bot commented Jul 17, 2023

Copy link

Copy Markdown

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

binarywang commented Jul 17, 2023

Copy link

Copy Markdown

Owner

感谢PR，不过建议以后不同功能独立建分支，分开提交PR

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

binarywang merged commit b6b80a2 into binarywang:develop

Jul 17, 2023

Molzx commented Jul 17, 2023

Copy link

Copy Markdown

Contributor Author

感谢PR，不过建议以后不同功能独立建分支，分开提交PR

👌，冇问题

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

sonatype-lift Bot reviewed

Jul 17, 2023

View reviewed changes

Comment thread

weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java

		* 提交购物订单接入审核
		*
		* @return WxOpenShoppingOrdersConfirmResult
		* @throws WxErrorException

sonatype-lift Bot Jul 17, 2023

Copy link

Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

11% of developers fix this issue

EmptyBlockTag: A block tag (@PARAM, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description.

Suggested change

	* @throws WxErrorException
	*

❗❗ 7 similar findings have been found in this PR

🔎 Expand here to view all instances of this finding

File Path	Line Number
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	107
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	64
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	73
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	55
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	82
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java	90
weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaService.java	696

Visit the Lift Web Console to find more details in your report.

ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command	Usage
`@sonatype-lift ignore`	Leave out the above finding from this PR
`@sonatype-lift ignoreall`	Leave out all the existing findings from this PR
`@sonatype-lift exclude <file\|issue\|path\|tool>`	Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
_{Click here to add LiftBot to another repo.}

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

sonatype-lift Bot reviewed

Jul 17, 2023

View reviewed changes

Comment thread

weixin-java-open/src/main/java/me/chanjar/weixin/open/api/WxOpenMaShoppingOrdersService.java

		import me.chanjar.weixin.open.bean.shoppingOrders.*;

		/**
		* @author xzh

sonatype-lift Bot Jul 17, 2023

Copy link

Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

31% of developers fix this issue

MissingSummary: A summary line is required on public/protected Javadocs.

ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command	Usage
`@sonatype-lift ignore`	Leave out the above finding from this PR
`@sonatype-lift ignoreall`	Leave out all the existing findings from this PR
`@sonatype-lift exclude <file\|issue\|path\|tool>`	Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
_{Click here to add LiftBot to another repo.}

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

sonatype-lift Bot reviewed

Jul 17, 2023

View reviewed changes

Comment thread

...n-java-open/src/main/java/me/chanjar/weixin/open/bean/result/WxOpenMaGetOrderPathResult.java

		import java.util.List;

		/**
		* @author xzh

sonatype-lift Bot Jul 17, 2023

Copy link

Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

31% of developers fix this issue

MissingSummary: A summary line is required on public/protected Javadocs.

ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command	Usage
`@sonatype-lift ignore`	Leave out the above finding from this PR
`@sonatype-lift ignoreall`	Leave out all the existing findings from this PR
`@sonatype-lift exclude <file\|issue\|path\|tool>`	Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
_{Click here to add LiftBot to another repo.}

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

sonatype-lift Bot reviewed

Jul 17, 2023

View reviewed changes

Comment thread

weixin-java-open/src/main/java/me/chanjar/weixin/open/api/impl/WxOpenMaBasicServiceImpl.java

		return WxOpenGsonBuilder.create().fromJson(response, WxOpenMaGetOrderPathResult.class);
		}

		private JsonArray toJsonArray(List<String> strList) {

sonatype-lift Bot Jul 17, 2023

Copy link

Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

14% of developers fix this issue

UnusedMethod: Private method 'toJsonArray' is never used.

ℹ️ Expand to see all @sonatype-lift commands

You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.

Command	Usage
`@sonatype-lift ignore`	Leave out the above finding from this PR
`@sonatype-lift ignoreall`	Leave out all the existing findings from this PR
`@sonatype-lift exclude <file\|issue\|path\|tool>`	Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file

Note: When talking to LiftBot, you need to refresh the page to see its response.
_{Click here to add LiftBot to another repo.}

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

binarywang modified the milestones: 4.5.4, 4.5.3

Aug 1, 2023

Reviewers

sonatype-lift[bot]

sonatype-lift[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

4.5.3

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.