Radicore Forum: RADICORE Suggestions » Message of the Day

Home » RADICORE » RADICORE Suggestions » Message of the Day

Show: Today's Messages :: Polls :: Message Navigator
E-mail to friend

Message of the Day [message #2034]

Mon, 04 May 2009 12:35

gpatti
Messages: 283
Registered: August 2008

Senior Member

Tony,

Thought it would be useful to have the ability to display a "Message of the Day" on the Home screen. This could be used to notify users of upcoming events / system changes, or to advise of significant changes following an upgrade etc.

Maybe this could be configured and set in the Menu Controls?

What do you think?

Graham

Report message to a moderator

Re: Message of the Day [message #2037 is a reply to message #2034]

Mon, 04 May 2009 15:42

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

Perhaps instead of having just a single message on the control table it might be more flexible to have a separate MOTD table with a separate record for each message. This would provide the opportunity to scroll though previous messages. I will see what I can do for the next release.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2039 is a reply to message #2034]

Mon, 04 May 2009 17:58

gpatti
Messages: 283
Registered: August 2008

Senior Member

Sounds good. May I suggest some kind of use of the start and end date functionality, so that a message can 'expire'. Perhaps only current messages are displayed automatically, whilst expired messages can be retrieved via scrolling etc? This would avoid having a very old message constantly displayed if it hasn't been superseded by a new one.

Report message to a moderator

Re: Message of the Day [message #2040 is a reply to message #2039]

Wed, 06 May 2009 06:50

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

Try the attached update and let me know what you think.

I've created a new database table, with associated tasks, for MOTD (message of the day). There is also a new screen which shows unexpired messages, latest first, immediately after passing through the logon screen. If there are no messages then the screen will not be shown.

Attachment: radicore-2009-05-06-update.zip
(Size: 126.29KB, Downloaded 2474 times)

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2041 is a reply to message #2034]

Wed, 06 May 2009 08:48

gpatti
Messages: 283
Registered: August 2008

Senior Member

At first look this is perfect and just what I had in mind. Couple of questions:
1. To display a message for a single day only I assume you enter the end date = start date. I noticed that due to a quirk with the current/historic properties, if I add a new motd for today only, it doesn't appear in the motd list unless I RESET the screen. It DOES, however, correctly appear in the Show screen so it's not an issue, just a bit confusing at first.
2. This functionality meets all my needs, but thinking ahead it might be appropriate to include the ability to filter which messages are shown by Role. The risk here is that it complicates what is nice and simple at the moment, but could be worth thinking about for the furture.

I've added some messages to start and end over the next few days, so I'll just keep an eye that it behaves as I'm expecting it will.

Great Work!

Report message to a moderator

Re: Message of the Day [message #2042 is a reply to message #2041]

Wed, 06 May 2009 09:34

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

I tried to duplicate your point #1, but I could not. I went into the 'List Message of the Day' screen which, by default, is set to display 'current' records only (where start_date<=today and end_date>=today), then I pressed 'New' to create a new message with today's date as both the start and end dates. When I returned to the 'List' screen it showed up as I expected.

As for your point #2, I can add role_id as an optional field on the mnu_motd table so that a message can be specific to a particular role or available to all roles.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2043 is a reply to message #2034]

Wed, 06 May 2009 10:12

gpatti
Messages: 283
Registered: August 2008

Senior Member

I've tried running through my debugger to see why I'm not getting todays messages listed, and I think I may have found a bug - please correct me if I'm wrong...

Lines 315 and 316 in std.table.class.inc show:
$search[$start_date] = "<='$date 00:00:00'";
$search[$end_date] = ">='$date 23:59:59'";

If I'm not mistaken the times on these two rows should be reversed, to become:
$search[$start_date] = "<='$date 23:59:59'";
$search[$end_date] = ">='$date 00:00:00'";

The start and end date fields in the mnu_motd table are defined as date fields rather than datetime fields, so I'm not sure why the times may be having an effect - maybe it's to do with different implementations of MySQL.

Regarding point 2 - this isn't something I need at the moment, but thought it might be a useful addition.

Graham

Report message to a moderator

Re: Message of the Day [message #2044 is a reply to message #2042]

Wed, 06 May 2009 10:21

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

The attached update adds 'role_id' as an optional field on the 'mnu_motd' table. The 'Show MOTD' screen will therefore be restricted to messages which are current and either linked to the user's role or are not linked to any role.

Attachment: radicore-2009-05-06-update2.zip
(Size: 18.89KB, Downloaded 2478 times)

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2045 is a reply to message #2043]

Wed, 06 May 2009 10:35

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

I think you are correct. I put that code in there for ORACLE users as the DATE data type includes the TIME whether you like it or not. I'm testing with MySQL 4.1 which didn't complain, so I didn't spot it. I assume you're using MySQL 5?

I'll fix this is the next release.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2046 is a reply to message #2034]

Wed, 06 May 2009 10:41

gpatti
Messages: 283
Registered: August 2008

Senior Member

If you want to let me know when you have a fix for that I'll test it against my database to see if it corrects the problem.

Report message to a moderator

Re: Message of the Day [message #2047 is a reply to message #2046]

Wed, 06 May 2009 11:02

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

The fix is exactly as you suggested, just reverse the times in lines 315 and 316 to the following:

Code: [Select all] [Show/ hide]

$search[$start_date] = "<='$date 23:59:59'";
$search[$end_date]   = ">='$date 00:00:00'";

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2048 is a reply to message #2034]

Wed, 06 May 2009 11:28

gpatti
Messages: 283
Registered: August 2008

Senior Member

Yes, I've tested, and that fixes the problem.

Although it isn't used in this case, the same code also appears in include.general.inc I assume it needs to be corrected here too.

Report message to a moderator

Re: Message of the Day [message #2049 is a reply to message #2048]

Wed, 06 May 2009 12:32

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

That's correct. I'll make the same change in my code.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2050 is a reply to message #2034]

Thu, 07 May 2009 06:37

gpatti
Messages: 283
Registered: August 2008

Senior Member

Tony,

I've noticed today that the Show task is displaying a message that should have expired (i.e. one that I set with start and end dates both as 2009-05-06).

Graam

Report message to a moderator

Re: Message of the Day [message #2051 is a reply to message #2034]

Thu, 07 May 2009 06:56

gpatti
Messages: 283
Registered: August 2008

Senior Member

Tony,

A further qualification on my previous message. The error is related to the role_id. If a message has a blank role_id then it IS excluded from the Show task. However, if a role_id is present it is not being excluded.

Graham

Report message to a moderator

Re: Message of the Day [message #2052 is a reply to message #2051]

Thu, 07 May 2009 10:05

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

In file menu.php can you change line 26 so that it reads:

Code: [Select all] [Show/ hide]

... AND (role_id IS NULL OR role_id='$role_id')";

Notice the parentheses around the two possible values for role_id.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2053 is a reply to message #2034]

Thu, 07 May 2009 10:23

gpatti
Messages: 283
Registered: August 2008

Senior Member

Done, and all now working correctly.
Thanks

Report message to a moderator

Re: Message of the Day [message #2054 is a reply to message #2034]

Thu, 07 May 2009 11:49

gpatti
Messages: 283
Registered: August 2008

Senior Member

I'm afraid I've discovered a new bug since the addition of the Role functionality (always a problem adding something as an afterthought!)

If you create a message with NO role id (i.e. for all roles) and then login with a role that has not been given access to the Show task (other than the Global role) the system tries to display the message and crashes, rather than ignoring the fact that you don't have access and not trying to display the screen.

I discovered this because I forgot to give task access!!

Report message to a moderator

Re: Message of the Day [message #2055 is a reply to message #2054]

Thu, 07 May 2009 12:45

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

You must give access to task 'mnu_motd(show)' to all roles, otherwise they won't be able to see that screen.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Re: Message of the Day [message #2056 is a reply to message #2055]

Thu, 07 May 2009 12:59

AJM
Messages: 2382
Registered: April 2006
Location: Surrey, UK

Senior Member

Run this SQL statement and it will insert entries into MNU_ROLE_TASK for all available roles:

Code: [Select all] [Show/ hide]

INSERT INTO mnu_role_task (role_id, task_id) 
     SELECT mnu_role.role_id, 'mnu_motd(show)'     
       FROM mnu_role 
      WHERE NOT EXISTS (SELECT 1 FROM mnu_role_task AS mrt 
                         WHERE mrt.role_id=mnu_role.role_id AND mrt.task_id='mnu_motd(show)');

I shall include this in the update script.

Tony Marston
http://www.tonymarston.net
http://www.radicore.org

Report message to a moderator

Previous Topic:	Auto numerate records in a list view
Next Topic:	PDF facility to produce letters

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

PDF

]

Current Time: Sat Jul 05 07:47:18 EDT 2025

Total time taken to generate the page: 0.20012 seconds