Skip to content
GitLab
Explore
Projects
Groups
Topics
Snippets
Projects
Groups
Topics
Snippets
/
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
MKCLOS
Core Development Platform
corepkgv2
Merge requests
!85
An error occurred while fetching the assigned milestone of the selected merge_request.
Fix: Revert coremongo panic fix.
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
Fix: Revert coremongo panic fix.
ab_Fix_RevertCoremongo
into
devbranch
Overview
0
Commits
1
Pipelines
0
Changes
1
Merged
Akshay Bharambe
requested to merge
ab_Fix_RevertCoremongo
into
devbranch
5 years ago
Overview
0
Commits
1
Pipelines
0
Changes
1
Expand
Fix: Revert coremongo panic fix as its breaking existing sls functionality. Need to check for empty arrays instead of empty string in existing code.
0
0
Merge request reports
Compare
devbranch
devbranch (base)
and
latest version
latest version
d6fea102
1 commit,
5 years ago
1 file
+
5
−
4
Expand all files
Preferences
File browser
List view
Tree view
Compare changes
Inline
Side-by-side
Show whitespace changes
Show one file at a time
dalmdl/coremongo/coremongo.go
+
5
−
4
Options
@@ -380,10 +380,11 @@ func (mg *MongoDAO) GetProjectedData(selector map[string]interface{}, projector
}
results
=
append
(
results
,
result
)
}
if
len
(
results
)
==
0
{
rs
:=
gjson
.
Parse
(
`[]`
)
return
&
rs
,
nil
}
// this fix is breaking existing sls functionality, need to review and change accrodingly.
// if len(results) == 0 {
// rs := gjson.Parse(`[]`)
// return &rs, nil
// }
ba
,
marshalError
:=
json
.
Marshal
(
results
)
if
errormdl
.
CheckErr2
(
marshalError
)
!=
nil
{
return
nil
,
errormdl
.
CheckErr2
(
marshalError
)
Menu
Explore
Projects
Groups
Topics
Snippets