Comments on: WP-Cat2Calendar

By: and

and — Tue, 09 Feb 2010 17:54:46 +0000

Thanks for experience sharing. I think it will be better just open someone’s plugin and look how it’s implemented there. I’m sure that someone already forced WP_Rewrite to work.

By: Camilo

Camilo — Tue, 09 Feb 2010 16:53:39 +0000

Hey!
I’ve also played with permalinks a few times, they can be quite a pain! I don’t think there is a plugin conflict here. Like you, I don’t fully understand how to play with the permalinks, but my (wild) guess is that you’re not making use of the filter “generate_rewrite_rules”. Here’s an example code I’ve used in a function:
global $wp_rewrite;


		$wp_rewrite->rules = array_merge(

			array( $this->slugUsergroup . '/(.+)/' . $this->slugHighlights . '/(.+)/?$' => 'index.php?' . $this->slugUsergroup . '=$matches[1]&' . $this->slugMethod . '=$matches[2]' )

			, $wp_rewrite->rules

		);

$wp_rewrite->rules = array_merge( array( $this->slugUsergroup . '/(.+)/' . $this->slugHighlights . '/?$' => 'index.php?' . $this->slugUsergroup . '=$matches[1]&' . $this->slugMethod . '=0' ) , $wp_rewrite->rules );
Oddly, I just noticed I never returned any value from this function (even though it’s a filter) and, believe me, I worked hard to get this one to work. Just look at the end of this page: http://codex.wordpress.org/Custom_Queries . Just be sure to add the rules to the beggining of the array, or you may have trouble when another rule matches before yours.
I guess the rules always have to be added, like everything else WP has to load before it spits something out. But, then again, maybe you should just flush when you save the options in your page or when WP calls it.
If you’d like a suggestion on a feature you could implement, you should consider looking at the hCalendar microformat ( http://microformats.org/wiki/hcalendar ). It’s ridiculously easy to implement, it’s an open standard and it’s a move towards semantic web!
Well, thank you for your time!

By: and

and — Fri, 05 Feb 2010 22:07:08 +0000

LOL, nice story about a bot

About a permalinks, I think you should update the plugin to latest version (if it’s not updated to 1.0.4 yet) and re-activate it (deactivate + activate). There was a problem with WP_Rewrite in an early versions. I’m using it on one of my websites along with other of my plugins which are uses permalinks too and there are no problems.

And also there can be any other plugins on your WordPress which uses permalinks not properly and just flushes the rules of cat2cal. Btw, I’m not sure that I’m using it properly but it works at least for two of my plugins which works on one WordPress installation. Documentation about a WP_Rewrite is not so clear and some of samples didn’t work properly for me when I tried to use it in a few of my plugins, there was just a conflict between them, one plugin flushed the rules of each other.

I’m not sure that it’s a good idea add a rules on every init but it works and if I’m adding it only on plugin activation then another plugin can be activated and just flush my rules so I’m not sure that everything so clear there.

I’ve rewritten wp_trim_excerpt function and future posts filter that you gave me, it’s already committed to plugin’s SVN. In addition, it won’t generate an excerpt for a password protected posts anymore.

I think this plugin will be better in the future. That’s great that you writing a responses, it’s very important for me. Thank you!

By: Camilo

Camilo — Fri, 05 Feb 2010 17:33:01 +0000

Hey, turns out I could work around “wp_cat2calendar_future_posts”. Just remove your filter and add another one with the hack (basic WP!). Anyway, here is a fix:
function wp_cat2calendar_future_posts($posts) { global $wp_query, $wpdb;


   if(is_single() && count($posts) == 0)

   {

      $_posts = $wpdb->get_results($wp_query->request);

      foreach ($_posts as $_p) {

      	  if ($_p->post_status == 'future') {

      	  	  $posts[] = $_p;

      	  }

      }

   }

return $posts; }

By: Camilo

Camilo — Fri, 05 Feb 2010 16:46:49 +0000

OK, thanks for everything!
It turns out my client preferred when titles were being shown, so I just worked the code to get it back the way it was before, but now showing a tooltip for the description when hovering over the post title.
Anyway, the only issue I can’t work around without changing your code is in “wp_cat2calendar_future_posts”, which (like I said earlier) exposes my drafts and pending review posts, so anyone who tries a bit of guessing (and is not logged in) can access them. I actually discovered this when a spam bot left a comment in one of those!
Thanks for the patience!

By: and

and — Fri, 05 Feb 2010 16:05:38 +0000

Thanks for explanation, it was obvious about wp_trim_excerpt, but anyway thank you. I just have not so much free time lately so when I’ll find a minute I fix this stuff. Actually I still can use wp_trim_excerpt because my plugin temporary overrides global variable $post when displaying posts, so I just should rewrite condition in my plugin to make it works like it should be.

By: Camilo

Camilo — Thu, 04 Feb 2010 14:42:42 +0000

Hello!
Actually, the images are displayed inside the calendar because these posts don’t have an explicit excerpt, so wp_trim_excerpt is called with the post content as a parameter. But wp_trim_excerpt doesn’t create a decent excerpt unless you pass an empty string to it and simulate being in the loop – it just applies a filter that nothing’s hooked to. That’s why I had to create a similar function without the “if ( ” == $text )” statement!
If you go to that website’s front page, you can see that everything in the “highlights” area has a “…” (post titles and their excerpts). I’ve used that modified wp_trim_excerpt to achieve this, works great!
I’m also not very fond of posts inside the table cells, I’m looking for a good tooltip jQuery plugin, but I still haven’t found a decent one! I’ll try to avoid making my own so I don’t have to mantain it just for this project (it’s not something I use a lot).

By: and

and — Thu, 04 Feb 2010 05:37:43 +0000

Hey Camilo! Yeah, I’ve seen that popup is cutted off when it’s too close to the sidebar on your website. (I just linked default.css to your page using firebug )

Also I’ve noticed that you’ve got an images in your calendar, looks like you’ve rewritten something because trim_excerpt should strip tags.

Well, I think so I can make a two options:

1. output only post titles like it was before.
2. output titles + post excerpts in popup. (I think I should also rework javascript)
3. any idea how to display it better?

But I really don’t like the old output method when the list of posts was inside a table cell, it looks ugly if u have a long titles or too much posts in a one day.

By: Camilo

Camilo — Thu, 04 Feb 2010 00:22:50 +0000

Hey Andrew! Glad to help, your plugin helps me!
Just to explain another thing about that modified function in my earlier comment: it’s the same function as wp_trim_excerpt, without the ‘if’ statement. If you pass a length of 20 to it, instead of making the 21st word be a ‘…’, it checks if the 21st is the last word and if it is, just use it instead of cutting it off (in case you were wandering).
Anyway, I removed my css to use the default bundled one and it was almost right, except that the “tooltip” was getting cut off (z-index wise) when too close to some elements with relative position (if I hover a day close to the top of the table, the top of the tooltip gets cut off; if I hover a day on the left, the left of the tooltip gets cut off). The website I implemented cat2cal is full of CSS positioning.
So I decided to re-adapt my old code instead, but the javascript implemented gets in my way, I can achieve something similar using CSS or implement my own javascript. Can you make it optional? I worked around it by using wp_deregister_script.

By: and

and — Wed, 03 Feb 2010 12:55:32 +0000

Camilo, thanks for your feedback!

I’ll check wp_trim_excerpt, I thought it’s old and stable function, shit happens
Looks like permalinks of future posts working only for admins or for post owner, I’m checking it now.

You should look to default.css to solve a problem of huge cells on your website because there should be a popups with events over the calendar, not a resizing cells. (see screenshot #1 on this page).