erp5_ods_style: fix the way to get form id
if i have a url like: https://erp5xxx/form_idxxx?relative_item_relative_url=item_module/7817
with the old way, form id will be 7817
-
-
@xiaowu.zhang , if I can imagine correctly this will introspect all object_view definitions and got "form_id" from there.
Most likely it should be OK as long as object_view definitions keep expected standard (i.e. what you change and hard -code).
I would try to use grep and find inside git if this is the case and if this is it no objects to merge in on my side.
-
-
we cannot use urlparse module in restricted python ? this would be a more "beautiful" way than parsing url like this.
-
If I understand correctly an action like this
string:${object_url}/Base_viewSomething?something=${here/getRelativeUrl}
triggers the bug ( I did not find any in nexedi/erp5.git , but it really seems a bug in this script, no reason not to support such actions )
-
If I understand correctly an action like this
string:${object_url}/Base_viewSomething?something=${here/getRelativeUrl}
yes, in some other project we have.
i find in nexedi/erp5.git, related document view
for this view, the url is something like:
which cause problem when make a report
-
we cannot use urlparse module in restricted python ? this would be a more "beautiful" way than parsing url like this.
no, i got error when access ParseResult's attribute
Site Error An error was encountered while publishing this resource. Error Type: Unauthorized Error Value: You are not allowed to access 'path' in this context
-
Most likely it should be OK as long as object_view definitions keep expected standard
i check all object_view in erp5, it's all in string:${object_url}/form_viewSomething format
so i think it's ok to merge
anyway, i'll wait for my test result
-
@xiaowu.zhang , thanks. Please merge if test passes.
-
merged
Toggle commit list
-
Please register or sign in to post a comment