Newscoop
Reviews
Recent reviews | Issues under review
CS-3353: include_path is not set properly and allow conflicting inclusions Previous commit contained...
:
CS-3353: include_path is not set properly and allow conflicting inclusions
Previous commit contained some errors. This is the fixed version.
CS-2359: Convert the Templates management to Zend/Doctrine
This issue includes the new themes functionality:
http://wiki.sourcefabric.org/x/RwBK
When refactoring take into account functionality for multiple outputs. See:
http://dev.sourcefabric.org/browse/CS-2607
Tree view:
http://www.dynamicdrive.com/dynamicindex1/treeview/index.htm
http://jquery.bassistance.de/treeview/demo/
http://www.jstree.com/demo
CS-2930: Windows support (Upgrade scripts do not work on Windows)
See this lengthy forum discussion:
http://forum.sourcefabric.org/index.php/mv/msg/3412/9688/#msg_9688
The upgrade scripts do not work on windows, though it should be possible for them to work.
CS-3082: Improve usability of geolocation tools in article edit screen
- Rename side menu from Locations > Geolocation (to improve consistency of communication)
Left hand menu in pop-up:
- Move city search field to top of left hand menu
- Give city search field a title: 'Search for city or co-ordinate'
- Change small right-facing arrow to a magnifying glass icon (to indicate search)
- Move country dropdown to below 'city search' field.
- Add title to dropdown: 'Specify country (optional)'
- Change styling on '+' and 'center city' - the look like buttons but aren't clickable
- Change '+' heading > Add location
- Add hover-over titles to list items (eg 'Add location to map' and 'Center city')
- Upon clicking add to map, clear list and search box
- Change wording from 'Disable > 'Clear' or 'Delete' (in fact doesn't disable duplicate the remove button? if so, delete it.)
- Remove latitude/longitude box and replace with 'point description field' (essential!)
Within edit location pop-up:
- Basic tab: add intro text: 'Name your location with a short label, url and full title.'
- Change title of 'Label' field to 'Location label'
- Label link? Is this a url? rename to 'Label url'.
- 'Short description for points listing' completely confusing. rename to 'Full locatiion title'
- Suggestion: Merge Text and Basic tabs.
- Text tabs - add small intro text: 'Change the description of this location.'
- Change radio button in text tab from 'view' to 'preview'
- Add label to Description field in Text tab: 'Location description'
- Change text in field 'fill in the point description' > 'Describe the location...'
- Image tab, add intro: Add an image to this location.
- Image tab: Can you take existing images from archive? Should be able to!
- Image tab - add sub heading: Resize image' (Q: does it crop or scale?)
- Video tab, add intro: 'Add a video to this location from YouTube, Vimeo or a Flash video.' - (I'm not sure about the flash aspect...)
- Video ID - what does this mean??? url? internal media active ID???
- Move source dropdown to above Video ID.
- Q: What do the flash dropdown fields do?
- Rename 'source' to 'Video source'
- Add subheading: 'Resize video' (Q: does it crop or scale?')
- Icon tab: add intro: Add an icon to your location.
- Change text 'add marker' to 'Add marker icon'
- Q: how do you upload your own icons? Surely this can be done here?
Map interface:
- Make base layer pop up appear automatically as open (not minimized)
- include title for map dimensions in top right. 'Map size'
- Make arrows into + and - icons rather than left and right (clicking left and right to increase up and down is counter-intuitive)
Text:
- Please change the following string: "The removal is from all language versions of the article" in the point removal popup to: "The point will be removed from all translations of the article".
CS-3159: User type should not be removed when users
User can remove a user type even though one or more users have been assigned that user type.
This is leading to data inconsistency because once the user type is removed, the users remain but also the relation to the (now non-existing) user type.
Expected behavior: System should not allow the user to remove the user type if assigned to any user and display the corresponding message to the user saying so.
To reproduce:
- Create new users and set the user type to one of the existing user types.
- Remove the user type.
- Notice the user type is removed. Users remain in the system. Check liveuser_groupusers table to see the relation is not removed.
CS-3159: User type should not be removed when users
User can remove a user type even though one or more users have been assigned that user type.
This is leading to data inconsistency because once the user type is removed, the users remain but also the relation to the (now non-existing) user type.
Expected behavior: System should not allow the user to remove the user type if assigned to any user and display the corresponding message to the user saying so.
To reproduce:
- Create new users and set the user type to one of the existing user types.
- Remove the user type.
- Notice the user type is removed. Users remain in the system. Check liveuser_groupusers table to see the relation is not removed.
CS-2334: Javascript-based Article Read Statistics
The current implementation of article read statistics does not work correctly with external caching (e.g. Squid, Varnish, Nginx). It also counts bots as legitimate reads.
More detail here: http://wiki.sourcefabric.org/display/CS/JavaScript-based+article+read+statistics
Please research if there is already something out there that does this before we reinvent the wheel.
CS-2329: Users should never lose their work
The user should never lose their work when they are editing a page.
Current idea:
Save the POST/GET variables to session when the user makes a request. If their session times out or if they are logged out in any way(for example, they click logout in a separate tab), they should be presented with a popup login form. It should tell them that their work is saved and they can resume editing once they log in again.
Coordinate with Andrey / Holman about this.