hoaivan / QuizMarker

0 stars 0 forks source link

code review #6

Closed hoaivan closed 12 years ago

hoaivan commented 12 years ago
  1. Khi thực hiện action thì check permission kiểu này: themeDisplay.getPermissionChecker().hasPermission(themeDisplay.getScopeGroupId(), WpConstants.PORTLET_NAME_STUDENT, (long)0, WpConstants.PERM_EDIT_STUDENT)) các permisson khai báo trong /src/resource-actions/default.xml => Chuwa lamf

=> Trong admin, làm gì có chỗ nào để set permission liên quan đến từng permission như EDIT_QUESTION, EDIT_EXERCISE,... Nên check quyền ko có ý nghĩa, lúc nào cũng true hết.

  1. Ko dung cau lenh specific to MySql. Thay = ANSI SQL cho FIND_IN_SET => DONE
  2. Add tag cho question => quay ra màn hình search chọn select để search theo tag thì ko thấy hiện ra. Trong select của question editor cũng ko hiện ra tag nào cả. Cả bên Exercise cũng teo kiểu này => Fix
  3. Save hay Publish xong question thì chỉ thấy hiện ra mỗi msg publish-question-sucessfully => có 2 hướng: (1) là về màn hình list, (2) ở màn hình hiện tại nhưng hiện đầy đủ các field + msg => Trước đây comment lại để debug. DONE
  4. Background image upload lên cho vào phía sau question editor => DONE
  5. Hiện ID của các Question Element để làm custom CSS cho tiện. Ở màn hình create new chưa có ID thì tạm hide ko sao, nhưng cần hiện ở màn hình Edit => DONE
  6. Hotspot vẫn chết, exception như hôm trước đã mail: An error occurred at line: 30 in the jsp file: /editor/questionform/hotspot.jsp The field Hotspot.imageId is not visible

27: /aui:input 28: /aui:input 29: /aui:input 30: /aui:input 31:

32: 33: /liferay-portlet:actionURL

=> Bây giờ ko xảy ra nữa.

  1. Set default msg cho 3 loại msg như msg có trong http://asiatrac.com/exercises/brainstorming/ => Mới để ra file langgue, chưa input làm.
  2. Trong màn hình Edit Exercise, cho button Back lên trên đầu và biến thành link ngang hàng tới title nhưng để right aligned. => DONE
  3. Tạo thử FIB thì gặp lỗi please-input-blank mặc dù nhập đủ dữ liệu cho place holder rồi => Bây giờ ko xảy ra nữa

10: FIB fill Image, ko cho các place holder trùng nhau về vị trí => DONE