Merge 937a8278a821fe517d22841f199b89a022ac732e into 1534abdb050acaa5aa6ef27f94e91bca97e6faa3

examples

@@ -1 +1 @@
-Subproject commit b57e4575ce971e6a6e581b1de0e77eccf61705c4
+Subproject commit f15e0c1978935dbe44ae6a7b8fe91de53abf1beb

net/ghttp/ghttp_request_param_file.go

@@ -19,9 +19,15 @@ import (
 	"github.com/gogf/gf/v2/internal/json"
 	"github.com/gogf/gf/v2/os/gfile"
 	"github.com/gogf/gf/v2/os/gtime"
+	"github.com/gogf/gf/v2/util/gconv"
 	"github.com/gogf/gf/v2/util/grand"
 )
 
+// init initializes the type converters for *UploadFile.
+func init() {
+	_ = gconv.RegisterTypeConverterFunc(stringToUploadFile)
+}
+
 // UploadFile wraps the multipart uploading file with more and convenient features.
 type UploadFile struct {
 	*multipart.FileHeader `json:"-"`
@@ -36,13 +42,18 @@ func (f UploadFile) MarshalJSON() ([]byte, error) {
 // UploadFiles is an array type of *UploadFile.
 type UploadFiles []*UploadFile
 
+// stringToUploadFile is a custom type converter for converting string to *ghttp.UploadFile.
+func stringToUploadFile(in string) (*UploadFile, error) {
+	return &UploadFile{}, nil
+}
+
 // Save saves the single uploading file to directory path and returns the saved file name.
 //
 // The parameter `dirPath` should be a directory path, or it returns error.
 //
 // Note that it will OVERWRITE the target file if there's already a same name file exist.
 func (f *UploadFile) Save(dirPath string, randomlyRename ...bool) (filename string, err error) {
-	if f == nil {
+	if f == nil || f.FileHeader == nil {
 		return "", gerror.NewCode(
 			gcode.CodeMissingParameter,
 			"file is empty, maybe you retrieve it from invalid field name or form enctype",

net/ghttp/ghttp_request_param_page.go

@@ -17,10 +17,12 @@ import (
 // GetPage creates and returns the pagination object for given `totalSize` and `pageSize`.
 // NOTE THAT the page parameter name from clients is constantly defined as gpage.DefaultPageName
 // for simplification and convenience.
+//
+// Deprecated: wrap this pagination html content in business layer.
 func (r *Request) GetPage(totalSize, pageSize int) *gpage.Page {
 	// It must have Router object attribute.
 	if r.Router == nil {
-		panic("Router object not found")
+		panic("router object not found")
 	}
 	var (
 		url            = *r.URL

net/ghttp/ghttp_z_unit_issue_test.go

@@ -19,6 +19,7 @@ import (
 	"github.com/gogf/gf/v2/net/ghttp"
 	"github.com/gogf/gf/v2/test/gtest"
 	"github.com/gogf/gf/v2/text/gstr"
+	"github.com/gogf/gf/v2/util/gmeta"
 	"github.com/gogf/gf/v2/util/gtag"
 	"github.com/gogf/gf/v2/util/guid"
 )
@@ -726,3 +727,30 @@ func Test_Issue4093(t *testing.T) {
 		t.Assert(client.PostContent(ctx, "/test"), `{"page":1,"pageSize":10,"pagination":true,"name":"john","number":1}`)
 	})
 }
+
+// https://github.com/gogf/gf/issues/4193
+func Test_Issue4193(t *testing.T) {
+	type Req struct {
+		gmeta.Meta `method:"post" mime:"multipart/form-data"`
+		File       ghttp.UploadFile `v:"required" type:"file"`
+	}
+	type Res struct{}
+	s := g.Server(guid.S())
+	s.BindMiddlewareDefault(ghttp.MiddlewareHandlerResponse)
+	s.BindHandler("/upload/single", func(ctx context.Context, req *Req) (res *Res, err error) {
+		return
+	})
+	s.SetDumpRouterMap(false)
+	s.Start()
+	defer s.Shutdown()
+	time.Sleep(100 * time.Millisecond)
+	// normal name
+	gtest.C(t, func(t *gtest.T) {
+		client := g.Client()
+		client.SetPrefix(fmt.Sprintf("http://127.0.0.1:%d", s.GetListenedPort()))
+		content := client.PostContent(ctx, "/upload/single", g.Map{
+			"file": "",
+		})
+		t.Assert(content, "{\"code\":51,\"message\":\"The File field is required\",\"data\":null}")
+	})
+}

util/gpage/gpage.go

@@ -5,6 +5,8 @@
 // You can obtain one at https://github.com/gogf/gf.
 
 // Package gpage provides useful paging functionality for web pages.
+//
+// Deprecated: wrap this pagination html content in business layer.
 package gpage
 
 import (
@@ -18,6 +20,8 @@ import (
 
 // Page is the pagination implementer.
 // All the attributes are public, you can change them when necessary.
+//
+// Deprecated: wrap this pagination html content in business layer.
 type Page struct {
 	TotalSize      int    // Total size.
 	TotalPage      int    // Total page, which is automatically calculated.
@@ -48,6 +52,8 @@ const (
 // /user/list/{.page}, /user/list/{.page}.html, /user/list?page={.page}&type=1, etc.
 // The build-in variable in `urlTemplate` "{.page}" specifies the page number, which will be replaced by certain
 // page number when producing.
+//
+// Deprecated: wrap this pagination html content in business layer.
 func New(totalSize, pageSize, currentPage int, urlTemplate string) *Page {
 	p := &Page{
 		LinkStyle:    "GPageLink",