liquid_feedback_frontend
view app/main/opinion/_action/update.lua @ 75:733f65c0c0a0
Bugfixes, feature enhancements, code-cleanup, and major work on API
Details:
- API
-- Allow relation name to be passed to helper function util.autoapi{...}
-- Added area API
-- Bugfixes in API
--- Correctly return initiatives (bug #162)
--- Correctly process "id" parameter for initiative API
--- Bugfix related to "state" parameter (bug #165)
--- Changed constant "discussion" to "accepted" (in model/issue.lua, used by API)
--- Fixed JSON encoding in auto_api (bug #181)
--- Ignore list filter "voted" in case of public access
--- Enable access to API without session
- Work on RSS feed (incomplete yet)
- Other bugfixes
-- Handle empty browser identification string
-- Handle invalid date in member/update.lua (bugs #24 #109 #115 #136)
-- Better handle errors while converting uploaded images. (bug #79 +5 duplicates)
-- Don't display revoked initiatives in list of new drafts (bug #134)
-- Fixed syntax error in app/main/member/_action/update_name.lua throwing unexpected error, when new name was too short
-- Do not display refresh support button for revoked initiatives
-- Repaired issue search (bug #150)
-- Fixed typos in german translation files
--- "initi(i)erte"
--- "Er(g)eignisse" (bug #161)
- Code cleanup
-- Removed deprecated motd files locale/motd/de.txt and locale/motd/de_public.txt
-- Removed redundant code in app/main/index/_updated_drafts.lua
- New features and (optical) enhancements
-- Support change of notify email; notification of not approved address added to start page
-- Settings dialog splitted into single pages
-- Mark deactivated members
-- Calendar for birthday selection in profile
-- Policy list public readable when public access is enabled
Details:
- API
-- Allow relation name to be passed to helper function util.autoapi{...}
-- Added area API
-- Bugfixes in API
--- Correctly return initiatives (bug #162)
--- Correctly process "id" parameter for initiative API
--- Bugfix related to "state" parameter (bug #165)
--- Changed constant "discussion" to "accepted" (in model/issue.lua, used by API)
--- Fixed JSON encoding in auto_api (bug #181)
--- Ignore list filter "voted" in case of public access
--- Enable access to API without session
- Work on RSS feed (incomplete yet)
- Other bugfixes
-- Handle empty browser identification string
-- Handle invalid date in member/update.lua (bugs #24 #109 #115 #136)
-- Better handle errors while converting uploaded images. (bug #79 +5 duplicates)
-- Don't display revoked initiatives in list of new drafts (bug #134)
-- Fixed syntax error in app/main/member/_action/update_name.lua throwing unexpected error, when new name was too short
-- Do not display refresh support button for revoked initiatives
-- Repaired issue search (bug #150)
-- Fixed typos in german translation files
--- "initi(i)erte"
--- "Er(g)eignisse" (bug #161)
- Code cleanup
-- Removed deprecated motd files locale/motd/de.txt and locale/motd/de_public.txt
-- Removed redundant code in app/main/index/_updated_drafts.lua
- New features and (optical) enhancements
-- Support change of notify email; notification of not approved address added to start page
-- Settings dialog splitted into single pages
-- Mark deactivated members
-- Calendar for birthday selection in profile
-- Policy list public readable when public access is enabled
| author | bsw | 
|---|---|
| date | Thu Jul 08 18:44:02 2010 +0200 (2010-07-08) | 
| parents | 8d91bccab0bf | 
| children | 6a12fb7e4963 | 
 line source
     1 local member_id = app.session.member.id
     3 local suggestion_id = param.get("suggestion_id", atom.integer)
     5 local opinion = Opinion:by_pk(member_id, suggestion_id)
     7 local suggestion = Suggestion:by_id(suggestion_id)
     9 if not suggestion then
    10   slot.put_into("error", _"This suggestion has been meanwhile deleted")
    11   return false
    12 end
    14 -- TODO important m1 selectors returning result _SET_!
    15 local issue = suggestion.initiative:get_reference_selector("issue"):for_share():single_object_mode():exec()
    17 if issue.closed then
    18   slot.put_into("error", _"This issue is already closed.")
    19   return false
    20 elseif issue.fully_frozen then 
    21   slot.put_into("error", _"Voting for this issue has already begun.")
    22   return false
    23 end
    27 if param.get("delete") then
    28   if opinion then
    29     opinion:destroy()
    30   end
    31   slot.put_into("notice", _"Your opinion has been deleted")
    32   return
    33 end
    35 if not opinion then
    36   opinion = Opinion:new()
    37   opinion.member_id     = member_id
    38   opinion.suggestion_id = suggestion_id
    39   opinion.fulfilled     = false
    40 end
    42 local degree = param.get("degree", atom.number)
    43 local fulfilled = param.get("fulfilled", atom.boolean)
    45 if degree ~= nil then
    46   opinion.degree = degree
    47 end
    49 if fulfilled ~= nil then
    50   opinion.fulfilled = fulfilled
    51 end
    53 opinion:save()
    55 slot.put_into("notice", _"Your opinion has been updated")
