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.

Sign up for GitHub

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

Jump to bottom

Object Builder Fixes #2

Merged
merged 2 commits into from
Jul 24, 2013
Merged

Object Builder Fixes #2

merged 2 commits into from
Jul 24, 2013
+30 −23

Conversation

rotema
Copy link
Contributor

@rotema rotema commented Jul 24, 2013

Resolved 2 bugs

  1. Base classes which doesn’t have custom_data field still tried to buildPkCriteria with the Custom_data md5 (and Fatal-ed out)
  2. When a partner had custom_data = NULL query failed (it md5(null) instead of going to the ‘else’ part)

@rotema
create update query with NULL correctly and fix builder for BaseClasses

Verified

This commit was signed with the committer’s verified signature. The key has expired.
addaleax Anna Henningsen
GPG key ID: D8B9F5AEAE84E4CF
Expired
Verified
Learn about vigilant mode
aeb9e87 
without custom_data field
@rotema rotema closed this Jul 24, 2013
@rotema rotema reopened this Jul 24, 2013
@rotema rotema merged commit a3d17c2 into IX-9.0.0 Jul 24, 2013
@flipmcf flipmcf mentioned this pull request Sep 6, 2013
Closed
assafb pushed a commit that referenced this pull request Nov 20, 2013
ps:P-171702 code review #2
8ea09fe
@jpluijmers jpluijmers mentioned this pull request Nov 3, 2014
Closed
@ElGabbu ElGabbu mentioned this pull request Nov 4, 2014
Open
@tan-tan-kanarek tan-tan-kanarek deleted the IX-9.0.0-ObjectBuilderFixes branch December 16, 2014 10:27
@rkreich rkreich mentioned this pull request Mar 22, 2015
Closed
@Patryk57 Patryk57 mentioned this pull request Jul 2, 2015
Closed
noam-arad added a commit that referenced this pull request Aug 2, 2015
@noam-arad
Add support for multiple kinds of hash funcitons to appToken service.…
2981142 
… Right now we support only sha1 and the addition will support also md5, sha256 and sha512.

PLAT-3477 #2 days
TaliHa pushed a commit that referenced this pull request Oct 27, 2015
Add getUrl action and update 'serve' action. Fix all code review rema…
038262c 
…rks by TanTan

branch Kajam-11.1.0-PLAT-3975-fix #2 hours
@TaliHa TaliHa mentioned this pull request Oct 27, 2015
Merged
TaliHa pushed a commit that referenced this pull request Nov 8, 2015
Fix bug in showGradeAfterSubmission,
f2c1d18 
Action clone of BaseEntry is now with a new parameter cloneOptions that defines the properties that should be copied in the clone operation. Currently only 'Categories' clone option is supported
PLAT-4058 #2 days
@TaliHa TaliHa mentioned this pull request Nov 8, 2015
Closed
danbar0kal added a commit that referenced this pull request Mar 3, 2021
@danbar0kal
Updating kZoomClient to use jwt - adjustments #2
1aaca76
danbar0kal added a commit that referenced this pull request Mar 21, 2021
@danbar0kal
Added source enum to KalturaCaptionAsset #2
799affd
amiras89 added a commit that referenced this pull request Aug 28, 2022
@amiras89
FOUN-627: Erans feedback #2
d8b9b14
amiras89 added a commit that referenced this pull request Aug 30, 2022
@amiras89
FOUN-607: Yossi feedback #2
8fd1a95
amiras89 added a commit that referenced this pull request Nov 17, 2022
@amiras89
FOUN-321: feedback #2
0bc05d9
amiras89 added a commit that referenced this pull request Dec 1, 2022
@amiras89
FOUN-321-zombiez: Eran is brilliant #2
70b0ec5
@enkosoftware enkosoftware mentioned this pull request Sep 1, 2023
Closed
2 tasks
amiras89 added a commit that referenced this pull request Sep 20, 2023
@amiras89
Yossi feedback #2
3153760
amiras89 added a commit that referenced this pull request Sep 27, 2023
@amiras89
Yossi feedback #2
b2b52d8
amiras89 added a commit that referenced this pull request Oct 2, 2023
@amiras89
Yossi feedback #2
4c28831
Amirhalfon1 added a commit that referenced this pull request Sep 4, 2024
@Amirhalfon1
remove spaces #2
5b56b26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@rotema