Changeset 350 in main


Ignore:
Timestamp:
09/28/11 14:00:18 (8 years ago)
Author:
pcosquer
Message:

remove useless request_dict variables

Location:
trunk/openPLM/plmapp
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/openPLM/plmapp/base_views.py

    r349 r350  
    331331    if hasattr(selected_object, "check_permission"): 
    332332        ctx["is_owner"] = selected_object.check_permission("owner", False) 
    333     return selected_object, ctx, request_dict.session 
     333    return selected_object, ctx 
    334334 
    335335coords_rx = re.compile(r'top:(\d+)px;left:(\d+)px;width:(\d+)px;height:(\d+)px;') 
    336336 
    337337def get_navigate_data(request, obj_type, obj_ref, obj_revi): 
    338     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    339     request.session.update(request_dict) 
     338    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    340339    FilterForm = get_navigate_form(obj) 
    341340    has_session = any(field in request.session for field in FilterForm.base_fields) 
  • trunk/openPLM/plmapp/views/main.py

    r349 r350  
    100100    .. include:: views_params.txt  
    101101    """ 
    102     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     102    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    103103     
    104104    object_attributes_list = [] 
     
    108108    ctx.update({'current_page' : 'attributes', 
    109109                'object_attributes': object_attributes_list}) 
    110     request.session.update(request_dict) 
    111110    return r2r('DisplayObject.htm', ctx, request) 
    112111 
     
    134133    .. include:: views_params.txt  
    135134    """ 
    136     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     135    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    137136    if request.method == 'POST': 
    138137        if request.POST["action"] == "DEMOTE": 
     
    151150                'is_signer' : is_signer,  
    152151                'is_signer_dm' : is_signer_dm}) 
    153     request.session.update(request_dict) 
    154152    return r2r('DisplayObjectLifecycle.htm', ctx, request) 
    155153 
     
    163161    .. include:: views_params.txt  
    164162    """ 
    165     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     163    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    166164     
    167165    if obj.is_revisable(): 
     
    172170        else: 
    173171            add_form = AddRevisionForm({"revision" : get_next_revision(obj_revi)}) 
    174     else: 
    175         add_form = None 
     172        ctx["add_revision_form"] = add_form 
    176173    revisions = obj.get_all_revisions() 
    177174    ctx.update({'current_page' : 'revisions', 
    178175                'revisions' : revisions, 
    179                 'add_revision_form' : add_form}) 
    180     request.session.update(request_dict) 
     176                }) 
    181177    return r2r('DisplayObjectRevisions.htm', ctx, request) 
    182178 
     
    190186    .. include:: views_params.txt  
    191187    """ 
    192     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     188    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    193189    history = obj.HISTORY.objects.filter(plmobject=obj.object).order_by('date') 
    194     history = history.values_list('date', 'action', 'details') 
    195190    ctx.update({'current_page' : 'history',  
    196                 'object_history' : history}) 
    197     request.session.update(request_dict) 
     191                'object_history' : list(history)}) 
    198192    return r2r('DisplayObjectHistory.htm', ctx, request) 
    199193 
     
    209203    .. include:: views_params.txt  
    210204    """ 
    211     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     205    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    212206     
    213207    if not hasattr(obj, "get_children"): 
     
    235229                'children': children, 
    236230                "display_form" : display_form}) 
    237     request.session.update(request_dict) 
    238231    return r2r('DisplayObjectChild.htm', ctx, request) 
    239232 
     
    249242    .. include:: views_params.txt  
    250243    """ 
    251     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     244    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    252245     
    253246    if not hasattr(obj, "get_children"): 
     
    263256    ctx.update({'current_page':'BOM-child', 
    264257                'children_formset': formset, }) 
    265     request.session.update(request_dict) 
    266258    return r2r('DisplayObjectChildEdit.htm', ctx, request) 
    267259 
     
    275267    .. include:: views_params.txt  
    276268    """ 
    277     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    278     request.session.update(request_dict) 
     269    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    279270     
    280271    if request.POST: 
     
    301292    .. include:: views_params.txt  
    302293    """ 
    303     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     294    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    304295     
    305296    if not hasattr(obj, "get_parents"): 
     
    324315                'parents' :  parents, 
    325316                'display_form' : display_form, }) 
    326     request.session.update(request_dict) 
    327317    return r2r('DisplayObjectParents.htm', ctx, request) 
    328318 
     
    337327    .. include:: views_params.txt  
    338328    """ 
    339     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     329    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    340330     
    341331    if not hasattr(obj, "get_attached_documents"): 
     
    352342                'object_doc_cad': obj.get_attached_documents(), 
    353343                'doc_cad_formset': formset}) 
    354     request.session.update(request_dict) 
    355344    return r2r('DisplayObjectDocCad.htm', ctx, request) 
    356345 
     
    365354    .. include:: views_params.txt  
    366355    """ 
    367     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    368      
    369     request.session.update(request_dict) 
     356    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     357     
    370358    if request.POST: 
    371359        add_doc_cad_form = AddDocCadForm(request.POST) 
     
    390378    .. include:: views_params.txt  
    391379    """ 
    392     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     380    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    393381     
    394382    if not hasattr(obj, "get_attached_parts"): 
     
    405393                'object_rel_part': obj.get_attached_parts(), 
    406394                'rel_part_formset': formset}) 
    407     request.session.update(request_dict) 
    408395    return r2r('DisplayObjectRelPart.htm', ctx, request) 
    409396 
     
    417404    .. include:: views_params.txt  
    418405    """ 
    419     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    420      
    421     request.session.update(request_dict) 
     406    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     407     
    422408    if request.POST: 
    423409        add_rel_part_form = AddRelPartForm(request.POST) 
     
    442428    .. include:: views_params.txt  
    443429    """ 
    444     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     430    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    445431 
    446432    if not hasattr(obj, "files"): 
     
    455441    ctx.update({'current_page':'files',  
    456442                'file_formset': formset}) 
    457     request.session.update(request_dict) 
    458443    return r2r('DisplayObjectFiles.htm', ctx, request) 
    459444 
     
    467452    .. include:: views_params.txt  
    468453    """ 
    469     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    470      
    471     request.session.update(request_dict) 
     454    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     455     
     456     
    472457    if request.POST: 
    473458        add_file_form = AddFileForm(request.POST, request.FILES) 
     
    494479    .. include:: views_params.txt  
    495480    """ 
    496     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     481    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    497482     
    498483    object_management_list = models.PLMObjectUserLink.objects.filter(plmobject=obj) 
     
    513498    ctx.update({'current_page':'management', 
    514499                'object_management': object_management_list}) 
    515     request.session.update(request_dict) 
     500     
    516501    return r2r('DisplayObjectManagement.htm', ctx, request) 
    517502 
     
    527512    :type link_id: str 
    528513    """ 
    529     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     514    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    530515    link = models.PLMObjectUserLink.objects.get(id=int(link_id)) 
    531516    if obj.object.id != link.plmobject.id: 
     
    543528    else: 
    544529        replace_management_form = SelectUserForm() 
    545     request.session.update(request_dict) 
     530     
    546531    ctx.update({'current_page':'management',  
    547532                'replace_management_form': replace_management_form, 
     
    559544    .. include:: views_params.txt  
    560545    """ 
    561     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     546    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    562547     
    563548    if request.method == "POST": 
     
    570555    else: 
    571556        add_management_form = SelectUserForm() 
    572     request.session.update(request_dict) 
     557     
    573558    ctx.update({'current_page':'management',  
    574559                'replace_management_form': add_management_form, 
     
    649634    """ 
    650635 
    651     obj, ctx, request_dict = get_generic_data(request) 
    652     request.session.update(request_dict) 
     636    obj, ctx = get_generic_data(request) 
     637     
    653638    if request.method == 'GET': 
    654639        if request.GET: 
     
    694679    .. include:: views_params.txt  
    695680    """ 
    696     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     681    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    697682    cls = models.get_all_plmobjects()[obj_type] 
    698683    non_modifyable_attributes = get_non_modifyable_attributes(obj, request.user, cls) 
     
    704689    else: 
    705690        modification_form = get_modification_form(cls, instance=obj.form_instance) 
    706     request.session.update(request_dict) 
     691     
    707692    ctx.update({'modification_form': modification_form, 
    708693                'non_modifyable_attributes': non_modifyable_attributes}) 
     
    723708    :return: a :class:`django.http.HttpResponse` 
    724709    """ 
    725     obj, ctx, request_dict = get_generic_data(request, "User", obj_ref) 
     710    obj, ctx = get_generic_data(request, "User", obj_ref) 
    726711    if obj.object != request.user: 
    727712        raise PermissionError("You are not the user") 
     
    734719    else: 
    735720        modification_form = OpenPLMUserChangeForm(instance=obj.object) 
    736     request.session.update(request_dict) 
     721     
    737722    ctx.update({'class4div': class_for_div, 'modification_form': modification_form}) 
    738723    return r2r('DisplayObject4modification.htm', ctx, request) 
     
    752737    if request.user.username=='test': 
    753738        return HttpResponseRedirect("/user/%s/attributes/" % request.user) 
    754     obj, ctx, request_dict = get_generic_data(request, "User", obj_ref) 
     739    obj, ctx = get_generic_data(request, "User", obj_ref) 
    755740    if obj.object != request.user: 
    756741        raise PermissionError("You are not the user") 
     
    764749    else: 
    765750        modification_form = PasswordChangeForm(obj) 
    766     request.session.update(request_dict) 
     751     
    767752    ctx.update({'class4div': "ActiveBox4User", 
    768753                'modification_form': modification_form}) 
     
    778763    .. include:: views_params.txt  
    779764    """ 
    780     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
     765    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    781766     
    782767    if not hasattr(obj, "get_object_user_links"): 
     
    785770    ctx.update({'current_page':'parts-doc-cad', 
    786771                'object_user_link': obj.get_object_user_links()}) 
    787     request.session.update(request_dict) 
     772     
    788773    return r2r('DisplayObjectRelPLMObject.htm', ctx, request) 
    789774 
     
    801786    :return: a :class:`django.http.HttpResponse` 
    802787    """ 
    803     obj, ctx, request_dict = get_generic_data(request, "User", obj_ref) 
     788    obj, ctx = get_generic_data(request, "User", obj_ref) 
    804789     
    805790    if not hasattr(obj, "get_user_delegation_links"): 
     
    811796    ctx.update({'current_page':'delegation',  
    812797                'user_delegation_link': obj.get_user_delegation_links()}) 
    813     request.session.update(request_dict) 
     798     
    814799    return r2r('DisplayObjectDelegation.htm', ctx, request) 
    815800 
     
    832817    :return: a :class:`django.http.HttpResponse` 
    833818    """ 
    834     obj, ctx, request_dict = get_generic_data(request, "User", obj_ref) 
     819    obj, ctx = get_generic_data(request, "User", obj_ref) 
    835820     
    836821    if request.method == "POST": 
     
    858843    elif role == "notified": 
    859844        role = _("notified") 
    860     request.session.update(request_dict) 
     845     
    861846    ctx.update({'current_page':'delegation', 
    862847                'replace_management_form': delegation_form, 
     
    881866    :return: a :class:`django.http.HttpResponse` 
    882867    """ 
    883     obj, ctx, request_dict = get_generic_data(request, "User", obj_ref) 
     868    obj, ctx = get_generic_data(request, "User", obj_ref) 
    884869     
    885870    if request.method == "POST": 
     
    901886        delegation_form = SelectUserForm() 
    902887    action_message_string = _("Select the user you no longer want for your \"%s\" role delegation :") % role 
    903     request.session.update(request_dict) 
     888     
    904889    ctx.update({'current_page':'parts-doc-cad', 
    905890                'replace_management_form': delegation_form, 
     
    922907    :return: a :class:`django.http.HttpResponse` 
    923908    """ 
    924     obj, ctx, request_dict = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    925      
    926     request.session.update(request_dict) 
     909    obj, ctx = get_generic_data(request, obj_type, obj_ref, obj_revi) 
    927910    if request.POST: 
    928911        checkin_file_form = AddFileForm(request.POST, request.FILES) 
     
    992975    return r2r('Navigate.htm', ctx, request) 
    993976 
    994  
     977@handle_errors 
    995978def display_users(request, obj_ref): 
    996     obj, ctx, request_dict = get_generic_data(request, "Group", obj_ref) 
     979    obj, ctx = get_generic_data(request, "Group", obj_ref) 
    997980    if request.method == "POST": 
    998981        formset = forms.get_user_formset(obj, request.POST) 
     
    1004987    ctx["user_formset"] = formset 
    1005988    ctx['current_page'] = 'users'  
    1006     request.session.update(request_dict) 
    1007989    return r2r("groups/users.htm", ctx, request) 
    1008990 
    1009  
     991@handle_errors 
     992def group_add_user(request, obj_ref): 
     993    """ 
     994    View of the *Add user* page of a group. 
     995 
     996    """ 
     997 
     998    obj, ctx = get_generic_data(request, "Group", obj_ref) 
     999    if request.method == "POST": 
     1000        form = SelectUserForm(request.POST) 
     1001        if form.is_valid(): 
     1002            obj.add_user(User.objects.get(username=form.cleaned_data["username"])) 
     1003            return HttpResponseRedirect("..") 
     1004    else: 
     1005        form = forms.SelectUserForm() 
     1006    ctx["add_user_form"] = form 
     1007    ctx['current_page'] = 'users'  
     1008    return r2r("groups/add_user.htm", ctx, request) 
     1009 
     1010 
Note: See TracChangeset for help on using the changeset viewer.