Roundcube Community Forum

 

Sieverules (managesieve) plugin

Started by zoc, May 09, 2009, 05:35:25 AM

Previous topic - Next topic

JohnDoh

hmmm i have had a chance to try it now, i made a few other changes though. try the repo version, not user if you need to removed the :addresses line or not from your example it worked for me with both.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

periferral

#106
when i add a new rule to filter by body, i get a message saying it is not supported by the server. im running dovecot 1.2.6 and managesieve server and it seems like filtering by body is supported. what gives?

periferral

@bambam i did see that but your options seems to be a little different from mine.
plugins/sieverules/config.inc.php

Mine is
$rcmail_config['sieverules_port'] = 2000;

Your seems to be
$sieverules_config['managesieve_port'] = 12000;

Maybe its nothing. but something to look into.

Also, what version of sieverules are you running?

When you start dovecot, do you see any errors? What does 'dovecot -n' return?

JohnDoh

#108
Quote from: periferral;23274when i add a new rule to filter by body, i get a message saying it is not supported by the server. im running dovecot 1.2.6 and managesieve server and it seems like filtering by body is supported. what gives?

there is a bug in how it writes the body rule. its fixed in the repo here: Commit 18155a8184960d9a4b023984143703e32f088589 to JohnDoh's RoundCube-Plugin-SieveRules-Managesieve - GitHub

Just incase there is any confusing about the config variables mentioned in the previous post. They have changed between the last released version and the repo version to fit in better with improvemnts to the RC plugin API.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

periferral

@johndoh. great. upgrading seems to have fixed the problem.

I have a couple of suggestions for this plugin if you would be so kind.

1. Currently the contents within div boxcontent have no margen and hence stick to the edges. Maybe a CSS addition of margin 15px all round would make this more attractive.

2. I quite like how the managesieve plugin (that is checked into roundcube) has the buttons on the top of the filters to add, delete, move up and move down. There are quite attactive rather than having the button below to add filters and the little up down arrows on the filter name and the delete is within the filter details.
   While this request is mostly for asthetic reasons, there is one functional element that is annoying in the current design. Lets say I want to move a filter up a few places. In the way it currently works, I select the filter I want to move and then press the up button. Now the rule has moved up, but I now need to move my mouse up to the new location to click the up button next to move it up again. By having the move up button outside, you just select the rule and click the move up button 3 times and its up 3 locations. No move moves required between moves. I hope this is clear.

3. This is something I'm not entirely sure how things work. If I select a non-default roundcube skin, it seems like I need to add stuff into plugins to add support for this. For example, I tried using mvision and I realized that mvision sources settings.css (when I click on the settings page) to get a consistent look between tabs etc. However, when I click the Filters tab, settings.css isn't sourced because it calls sieverules. I would have thought that roundcube should have sourced settings.css before handing in control to the plugin. But I don't know if there is a consitent rule. Again this is not something you need to do, but I'd like to know more about it.

I can code but php isn't something I'm comfortable with but let me know if you need any help with the plugins.

JohnDoh

Quote from: periferral;233471. Currently the contents within div boxcontent have no margen and hence stick to the edges. Maybe a CSS addition of margin 15px all round would make this more attractive.

I cannot see where exactly you are talking about. What browser are you talking about? Could you attach a screenshot please?

Quote from: periferral;233472. I quite like how the managesieve plugin (that is checked into roundcube) has the buttons on the top of the filters to add, delete, move up and move down. There are quite attactive rather than having the button below to add filters and the little up down arrows on the filter name and the delete is within the filter details.
   While this request is mostly for asthetic reasons, there is one functional element that is annoying in the current design. Lets say I want to move a filter up a few places. In the way it currently works, I select the filter I want to move and then press the up button. Now the rule has moved up, but I now need to move my mouse up to the new location to click the up button next to move it up again. By having the move up button outside, you just select the rule and click the move up button 3 times and its up 3 locations. No move moves required between moves. I hope this is clear.

I don't like the buttons at the top, its one of the reasons I made my own version so I wont be adding those. In my opinion the bottons at the top dont fit in with the style of the other settings tabs in the core. About the moving, you dont have to press the arrows, you can drag 'n drop.

Quote from: periferral;233473. This is something I'm not entirely sure how things work. If I select a non-default roundcube skin, it seems like I need to add stuff into plugins to add support for this. For example, I tried using mvision and I realized that mvision sources settings.css (when I click on the settings page) to get a consistent look between tabs etc. However, when I click the Filters tab, settings.css isn't sourced because it calls sieverules. I would have thought that roundcube should have sourced settings.css before handing in control to the plugin. But I don't know if there is a consitent rule. Again this is not something you need to do, but I'd like to know more about it.

The plugin was made for the default skin. The skin documentation is still being worked on so right now it is impossible to know what should always be in the skin and what should be in the plugin.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

periferral

@johndoh
Here is the original screenshot
http://dl.dropbox.com/u/163717/roundcube/rcsieve_orig.jpg

Here is after I added this to the skins/default/sieverules.css file
.boxcontent {
  margin: 15px;
}
http://dl.dropbox.com/u/163717/roundcube/rcsieve_margin.jpg

JohnDoh

the boxcontent class is part of the default skin, it is setup in common.css. the sieverules screen will look the same as the settings screen. what browser are you using?

about the links: there is a link to my plugins page in the first post in this thread, and in my sig and on the RC plugins page. my page as links to the repos as well as the translator, screenshots, changelog etc. I am sorry you had to search for the files but I am not sure what more I can do.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

periferral

@johndoh
I'm using firefox 3.5.5

When you say default skin, you mean default skin for sieverules? boxcontent isn't a part of roundcube. For exameple, when I click on the preference tab, I see that the contents are in a div called prefs-details which has a margin of 15px.

Seems like boxcontent is a part of the sieverules default skin and hence should be a part of sieverules.css.

I'm not sure if what you are saying is correct. Looking at the code for the plugin, I see this

plugins/sieverules/skins/default/templates/editsieverule.html:

periferral

i had the debian version of roundcube. I checked out trunk and the border padding is no longer an issue.

JohnDoh

There were some posts in this thread about connection issues with the managesieve plugin available in the RoundCube trunk. These posts have been moved to a new thread here http://www.roundcubeforum.net/api-based-plugins/5860-managesieve-connection-problems.html

I would like to keep the two seperate to minimse any confusion between these 2 plugins.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

kapeka

Hi;


I've managed to get sieverules working with a cyrus imap server, but still have a pb.

A vacation message for the real user ([email protected]) is send in reply to the sender, as expected.
But it does not work with aliases (like [email protected], which is an alias user for [email protected]).
I expected that the alias field for addresses while adding a vacation rule can be filled with alias addresses, but I can't edit that field.

Any hints other than the advanced editor to edit the sieve rules? I can do, but I'm afraid my users can't :)

TIA kp

JohnDoh

if you tick the "advanced options" box under the message box you can select aliases which should be used. the email address has to be one of your identies for it to show up in this list.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

kapeka

Thx, adding an identity for (every) alias did the trick.
A solution that can be explained to users...

kapeka

Rando

Hi JohnDoh,
thanks again for this usefull plugin.

I wonder if there is an easy way to configure sieverules to add new filters at the top of the list ? or maybe only for vacation filters ?

Thanks !
R.