-
Notifications
You must be signed in to change notification settings - Fork 92
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
Removed o_
prefixes
#379
Removed o_
prefixes
#379
Conversation
@@ -5,7 +5,7 @@ | |||
<th class="table-id-column">#</th> | |||
<th width="80px"> | |||
<div class="pos"> | |||
<a class="sorting{{ cmf_currentOrder('o_id') }}" href="{{ path('customermanagementframework_admin_customers_list', cmf_nextFormOrderParams('o_id')) }}"> | |||
<a class="sorting{{ cmf_currentOrder('id') }}" href="{{ path('customermanagementframework_admin_customers_list', cmf_nextFormOrderParams('id')) }}"> |
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.
@dvesh3 @mcop1 also on twig files same thing as https://github.com/pimcore/customer-data-framework/pull/379/files#r1039488239
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.
we should pass this variable (with value from getVersionDependentDatabaseColumnName
) from list action and read it here..
SELECT `breaksInheritance` FROM `plugin_cmf_segment_assignment` WHERE `elementId` = elementIdent AND `elementType` = 'object' INTO breaks; | ||
|
||
WHILE (elementExists = 0 OR breaks IS NULL OR breaks <> 1) AND elementIdent > 1 DO | ||
SELECT `parentId` FROM `objects` WHERE `id` = elementIdent INTO elementIdent; |
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.
@dvesh3 what about here?
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.
I think we have to check column here and run command specifically. please see suggestions here https://stackoverflow.com/questions/133031/how-to-check-if-a-column-exists-in-a-sql-server-table
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.
not sure though if this is the best solution
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.
@aashan10 please tests are still failing, most probably due to stored procedure change. thanks!
|
Closed in favor of #382 |
* Changes for #379 * Changes for #379 * Removde o_ prefix Signed-off-by: dvesh3 <divesh.pahuja@pimcore.com> * Removde o_ prefix Signed-off-by: dvesh3 <divesh.pahuja@pimcore.com> * Removde o_ prefix Signed-off-by: dvesh3 <divesh.pahuja@pimcore.com> Signed-off-by: dvesh3 <divesh.pahuja@pimcore.com> Co-authored-by: Ashan Ghimire <ashanghimire10@gmail.com>
Resolves #342