Very nice module..!

I have a question however: How do you use/apply Image Styles on the user's picture, both in the stream and the comments? Some users upload pictures that need to be scaled and/or cropped and I did not see an option to do that (couldn't see it in a View either.)

Thank you. :)

CommentFileSizeAuthor
#12 fbsmp_imagecache.patch4.24 KBDinesh Kumar Sarangapani
#9 fbsmp_imagecache.patch4.23 KBDinesh Kumar Sarangapani
#6 Statuses.JPG18.51 KBTriumphent

Comments

icecreamyou’s picture

icecreamyou commented
Project: Statuses (Social Microblog) » Facebook-style Micropublisher
Category: support » bug

You can set this in the photo plugin configuration, but it was not correctly ported from Drupal 6 (it still claims to rely on ImageCache, which is now the core Image module in D7).

Triumphent’s picture

Triumphent commented

Thank you. Unfortunately, an unresolved error during the installation of the photo plugin, makes the Statuses Module unusable:
Fatal error: Class 'rules_data_type' not found in C:\wamp\www\sites\all\modules\fbsmp\fbsmp.rules.inc on line 127

Triumphent’s picture

Triumphent commented

How difficult would it be to replace the Status: User Picture field in the View with the User: Picture field?
I have tried it and it works but the vertical alignment is off as it puts it in a different column (table based.) Any suggestion on this? Thanks.

icecreamyou’s picture

icecreamyou commented

As you've noticed, #1388612: Rules integration is broken addresses that error...

The Statuses: User Picture field is almost exactly the same as the User: Picture field except that it fixes the alignment issue. It's actually just a CSS thing in your theme (typically it's to float the user's picture on their profile to the right) so it's easy to override.

Triumphent’s picture

Triumphent commented

Ahhh... Thank you. I didn't think about the theming thing. It should be easy to fix. Thanks for your help. :)

Triumphent’s picture

Triumphent commented
StatusFileSize
new Statuses.JPG18.51 KB

I have deleted the Status: Sender Picture that could not accept scaling and cropping. Changing the CSS only allows scaling but not cropping. Instead, I have added the User Picture field, thereby effectively creating two columns. And it is great!
I also had to delete the "FEED" tab because of the missing style plugin that preempted the preview.
Now, there is this "[sender_pic]" that replaces the actual Status: Sender Picture (see attached pic) that I would like to eliminate as well. What do you suggest the best course of action is (without too much hacking of a great module if at all possible?) Thank you.

icecreamyou’s picture

icecreamyou commented

@Triumphent: Please keep each issue to one topic. You can open a new issue for other problems if you would like (though the answer to your question is you should edit the last field in the view and remove the [sender_pic] token from its settings).

Triumphent’s picture

Triumphent commented

@IceCreamYou: Sorry about that. Will be more careful next time. And thanks for the tip, it works great..!

Dinesh Kumar Sarangapani’s picture

Dinesh Kumar Sarangapani commented
Status: Active » Needs work
StatusFileSize
new fbsmp_imagecache.patch4.23 KB

Here is a Patch for Image Cache issue.

icecreamyou’s picture

icecreamyou commented 
+++ b/plugins/photo.inc
@@ -431,11 +431,12 @@ function theme_fbsmp_photo_attachment_imagecache($variables) {
+  $attributes = '';

This should be $attributes = array();

Other than that the patch in #9 looks good to me. Any testers?

bossmac’s picture

bossmac commented

doing fine :) thanks for the patch

Dinesh Kumar Sarangapani’s picture

Dinesh Kumar Sarangapani commented
StatusFileSize
new fbsmp_imagecache.patch4.24 KB

@IceCreamYou and @bossmac thank you...

I have attached the modified patch

mathankumarc’s picture

mathankumarc commented

works great :)

+++ b/plugins/photo.incundefined
@@ -431,11 +431,12 @@ function theme_fbsmp_photo_attachment_imagecache($variables) {
-    list($presetname, $style) = split(':', $settings['imagecache_preset']);
+   list($presetname, $style) = explode(':', $settings['imagecache_preset']);

split has been deprecated from PHP 5.3.0 Nice work!

mathankumarc’s picture

mathankumarc commented
Status: Needs work » Reviewed & tested by the community

changing the status.

icecreamyou’s picture

icecreamyou commented
Status: Reviewed & tested by the community » Fixed

Great, thank you. Committed #12 to dev!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.