summaryrefslogtreecommitdiff
path: root/modules/comment
AgeCommit message (Collapse)Author
2013-01-31Drop the requirement to have the install() function set the moduleBharat Mediratta
version. It's redundant. Fixes #1985.
2013-01-22gallery.menalto.com -> galleryproject.orgBharat Mediratta
codex.gallery2.org -> codex.galleryproject.org Fixes #1957.
2013-01-21Update copyright to 2013. Fixes #1953.Bharat Mediratta
2012-10-08Installer for comment module version 6 still set rss_visible to "both"Bharat Mediratta
not "all". Fix that now and rebuild the installer package. Fixes #1917.
2012-08-02Fix up per item feed urls. Fixes #1906.Bharat Mediratta
2012-05-07Oops, fix up a bug originally added inBharat Mediratta
7d66ab2e949bc915f108737f08cac2f9057ef729 when I tweaked the name of the rss_available variable to be rss_visible, but got it wrong. Bump the comment module to 6 so that we run the installer and clean up old vars. Fixes #1854.
2012-05-07Update date format to make it RFC822 compliant. Thanks to moullas. Fixes ↵Bharat Mediratta
#1798.
2012-03-29Improve comment RSS feed visibility, initially added by Thomas E. Horner inBharat Mediratta
fc942aacda07346fa9af04853659eaeac1e766d3. Change some variable names, refactor out visibility checking code, actually check visibility at generation time instead of just suppressing the UI, update module.info Fixes #1829.
2012-03-29Merge commit 'fc942aa'Bharat Mediratta
2012-02-27Update copyright to 2012. #1822Bharat Mediratta
2011-07-08availability of rss is now configurableThomas E. Horner
2011-04-26Overhaul the Admin > Content > Comments page. Use a tabbed interfaceBharat Mediratta
for consistency with other parts of the admin UI (ie: G2 import). The UI is now Ajax driven because that's how jQuery tabs works best in this case. This requires a little finagling in JS to get the paginator links to load properly. While I'm in there, add a paginator at the top of the table, make the buttons in the delete tab sane and smooth scroll back to the top of the tab when switching pages. Fixes #1702
2011-04-26Stop using Pagination() and instead use $theme->pager() in views.Bharat Mediratta
Move the pager() function up to Gallery_View and replace themes/admin_wind/views/pager.html.php (Pagination based) with a modified version from the wind theme in themes/admin_wind/views/paginator.html.php. Fixes #1718.
2011-04-23Oops, fix broken codex urls. For #1698.Bharat Mediratta
2011-04-23Oops, this is the rest of the modules and themes for #1696 and #1698.Bharat Mediratta
2011-03-27A little more work on kandsten's fix for IPv6 support inBharat Mediratta
10785b1e820c5e10d982c6b49125903886f7b889: - some style cleanup - bump the module version in module.info - rebuild the installer.sql
2011-03-27Extend comment module field lenghts to fit IPv6 remote host addresses and ↵Kriss Andsten
long (but legally so) hostnames.
2011-01-21Update copyright to 2011.Bharat Mediratta
2011-01-15Fix all the head() and admin_head() theme callbacks to return theBharat Mediratta
results of the $theme->css() and $theme->script() calls. This handles the case where combining scripts/css returns HTML instead of putting it in the queue for combination. Fixes #1611.
2010-12-28Use db::expr instead of "new Database_Expression". Resolves #1560.Bharat Mediratta
2010-10-25All modules should be named Xxx_Core for extensibility. Fixes #1458.Bharat Mediratta
2010-09-18Fix a bug in the way that we calculate expiration dates for oldBharat Mediratta
comments by using Database_Expression. Fixes ticket #1386.
2010-09-16Convert the guest_url ORM validation failure back to the url formBharat Mediratta
parameter and put in a message requiring a valid url. Fixes ticket
2010-09-04Use the title of the root album as the site title for all RSS feeds.Bharat Mediratta
Fixes ticket #1307.
2010-08-22Add "captcha_protect_form" event that the recaptcha module grabs andBharat Mediratta
uses to add a captcha to the end of the first group in the form. If there are no groups, it adds the captcha at the end of the form. Updated user_profile and comment forms to use it.
2010-08-09Fix a pagination bug. Fixes ticket #1280.Bharat Mediratta
2010-08-08Use the gallery helper date/time functions wherever we format dateBharat Mediratta
time for the browser. Fixes ticket #1278.
2010-08-08Convert tabs to spaces to fix File_Structure_Test.Bharat Mediratta
2010-07-31Full pass over all the JSON encoding and JS dialog code. We now abideBharat Mediratta
by the following rules: 1) An initial dialog or panel load can take either HTML or JSON, but the mime type must accurately reflect its payload. 2) dialog form submits can handle a pure HTML response, but the mime type must also be correct. This properly resolves the problem where the reauth code gets a JSON response first from the reauth code, and then an HTML response when you reauth and continue on to a given form -- try it out with Admin > Settings > Advanced. 3) All JSON replies must set the mime type correctly. The json::reply convenience function does this for us. 4) By default, any HTML content sent back in the JSON response should be in the "html" field, no longer the "form" field. The combination of these allows us to stop doing boilerplate code like this in our controllers: // Print our view, JSON encoded json::reply(array("form" => (string) $view)); instead, controllers can just return HTML, eg: // Print our view print $view; That's much more intuitive for developers.
2010-07-31More patches as part of #1225. Change the 'core' modules to use the json::replyTim Almdal
method to set the content type header and encode the response as a json object
2010-06-20Rename the feed variable from "children" to "comments" since thatBharat Mediratta
makes more semantic sense.
2010-06-20Oops. Remove debug line.Bharat Mediratta
2010-06-20Simplify the descendent logic. viewable() already joins with theBharat Mediratta
items table so there's no need for a subquery. The subquery could generate way too many ids since it didn't pay attention to permissions. This isn't a security problem since we were restricting the item ids according to permissions in the outer query, but it's wasteful.
2010-06-18Undo "else" clause -- we should keep the logic simple and easy toBharat Mediratta
follow, even if it's redundant. Expand a <ul> to multiple lines.
2010-06-17Fix for ticket #1163. Don't all guests to a comment when there are no ↵Tim Almdal
comments and the comment access permission is register users.
2010-06-16Remove comment_rest::post() -- that's in comments_rest now.Bharat Mediratta
2010-06-16Add REST support for comments. You can view, add, edit and deleteBharat Mediratta
comments.
2010-06-16slap my wrist... i forgot a debugging statementTim Almdal
2010-06-16Fix for ticket #1131. If the rss feed is for an item, then retrieve the ↵Tim Almdal
item. Using the left and right pointers find all the comments for the child items. Thanks to jankoprowski for the initial investigation.
2010-06-15Fix a bug in the upgrader where we weren't bumping the version numberBharat Mediratta
during the upgrade path, so the 2nd stanza (version 2 to version 3) was never getting executed.
2010-06-05Create a UI under Admin > Settings > Comments where you can limitBharat Mediratta
comments only to registered users. It's simplistic, but is better than adding a permission since generally this setting will be used Gallery-wide. Fixes ticket #1002
2010-05-16Rename the admin_comments view to admin_manage_comments to match the controller.Bharat Mediratta
2010-05-16Rename admin/comments to admin/manage_comments to make room forBharat Mediratta
admin/comments to be a settings page.
2010-05-14Require a well-formed email address for all comments.Bharat Mediratta
2010-04-30Add page_title to admin views. Closes #1038.ckieffer
2010-04-21Comment updates. Add empty <li> to empty <ul> for HTML validation, fixes ↵Chad Kieffer
#1066. Minor CSS/JS fixes. Add scrollTo effect when clicking on 'Add a Comment'.
2010-03-03Update the copyright to 2010. It's only 3 months into the year :-)Bharat Mediratta
2010-02-23Reset the active user to admin in all test cases where we change theBharat Mediratta
user to something else.
2010-02-18Don't need 'right' class here. Positioning is absolute to the right.Chad Kieffer
2010-02-14HTML validation, avoid empty <ul>Andy Staudacher