CVS log for loncom/interface/lonmsg.pm

Up to [LON-CAPA] / loncom / interface

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

- For 2.11.5 (modified)
  Include changes in 1.249

- For 2.11.X
  Backport 1.250, 1.251

- Typo

- For users with srm priv, unless replies are explicitly disallowed, set
  a Reply-to email address, where available, in case the "From" address is
  reserved for use by TLS authentication.

- For 2.11.4 (modified)
  Include changes in 1.249

- URLs in direct links to LON-CAPA messages in notification email, and to
  publicly accessible syllabus or portfolio files can be set to use hostname
  of portal server instead of user's homeserver as first part of URL.

- For 2.11
  Backport 1.248

Bug 6955
 -  If IP-based blocking set in domain config is in effect for user's
    current IP address, and 'Messaging 'is among blocked functions:
   (a) 'Send Feedback' (in course context), for resource(s) which allow it
       is only messaging mechanism available to users with out 'evb' priv.
   (b) Subject and Content for messages sent to instructor(s) via 'Send Feedback'
       are replaced in user's Sent Folder with: "Not shown due to IP block".

- For 2.11
  Backport 1.247

- Bug 6955 IP-based blocking. Pass user'sIP address as third arg to
  loncommon::blockcheck() and second to loncommon::blocking_status().

- For 2.11
  Backport 1.246

- Retrieval of requestor's IP address centralized in lonnet::get_requestor_ip()
- Domain configuration to allow domain's LON-CAPA nodes to operate behind a
  WAF/Reverse Proxy using aliased hostname (CNAME).
- Web requests from other nodes bypass the WAF as their requests are made
  directly to the server hostname (A record); same for internal LON-CAPA
  connections for lonc -> lond.

- Provide 'Reply To' address set in lonmsg.pm when calling mime_email() in
  loncommon.pm
- Update documentation for mime_email().

- Call Apache::lonnet::hostname($lonhost) before accessing
  $Apache::lonnet::protocol{$lonhost) so %Apache::lonnet::protocol is populated.

- For 2.11
  - Backport 1.243

- speedup "What's New" page for people with large number of messages in INBOX.

New function to send email (with an attachment)

loncommon.pm contains &mime_email() to build and send emails using MIME::Lite.  The
functionality is based on the previous code in lonsupportreq.pm.  Now both
lonsupportreq.pm and lonmsg.pm call mime_email().

Files attached to LON-CAPA messages (normal or critical) will now be included in the
external email if the "Send copy to permanent e-mail address (if known)" option is
selected.  Attachments were previously only sent internally and stripped from the
external email.

This should resolve bug #6723.

Reverting to rev. 1.239

MIME::Entity is not included with LON-CAPA.
Will change to MIME::Lite per Stuart Raeburn's suggestion.

Attaching files to external email

Files attached to LON-CAPA messages (normal or critical) will now be included in the
external email if the "Send copy to permanent e-mail address (if known)" option is
selected.  Attachments were previously only sent internally and stripped from the
external email.

The module Mail::Send was replaced by MIME::Entity to allow attachments.

This should resolve bug #6723.

- Coding style: eliminate $_ for readability.

- Report if e-mail was created, in case where privileged user checked:
  "Send copy to permanent e-mail address (if known)". Also log this.

- Coding style: keys()

- Coding style.

Corrected wording typos
Updated related phrases in German translation file

Removal of "priv" more global: now done in lonnet::declutter
(related to BZ 1320, 6591)

Follow up lonpubdir.pm 1.143:
- Bug 1320: Corrected URL matching to decide whether bombs should be displayed or not in CSTR.
Bomb link in resource row shows list of bombs again now.

- Bug 6413.
  - Subject for notification e-mail uses subject from corresponding LON-CAPA
    message (uness blank), with [LON-CAPA] prepended.

- Use a more suitable name for this variable in %env.

Bug #5528:
Explicitly send character encoding UTF-8 in outgoing e-mail to ensure correct character display in user's e-mail client:
- Notification e-mails
- Broadcast e-mails (DC)
- Helpdesk requests

Character encoding of special chars in e-mail subject still dysfunctional (as before)

- Handle course feedback recipients stored with pre 2.4 LON-CAPA:
  (a) @ was acceptable separator (in place of :) for username:domain.
  (b) textbox entry for comma-separated list of recipients.

Bugfix:
Allow students to send course feedback to course personnel who have a username which contains a "@" character.

Background:
"@" in usernames were replaced by ":" in the sub routine "secapply" since a change in lonfeedback.pm 1.36 to address bug #728. The sub routine moved lateron with version 1.242 to lonmsg.pm 1.197.

The replacement was probably made to ensure the user made entries to follow the format "username:domain". "username@domain" was replaced by "username:domain". Unfortunately, "username@institution:domain" was incorrectly replaced by "username:institution:domain".

The current course configuration doesn't allow free entries like "username:domain". The username is entered, but the domain is chosen from a dropdown list. The "@"->":"-replacement isn't needed anymore. courseprefs.pm takes care about this. Removing the replacement allows to receive feedback again for "@"-users.

- Backport 1.227.

- Fix typo.

- Backport 1.226.

- Improved more general fix for bug6233.
  Unless recipient user preference set to retain HTML tags in notification e-mail:
     - Remove HTML tags and decode HTML Entities in message excerpt, then encode in utf8 before sending e-mail.

Fixed bug #6233, non-breaking page space tags stripped from outgoing external messages.

Update the display for new Messages in the toplevelmenu, if you go on the message site the display in the toplevelmenu now is cleard

Add new function mynewmail to check if new mails are aviable(for the toplevel menu)

- Move &user_lang() from lonmsg.pm to loncommon.pm so it is more generally available.
- This routine is used to get a language handle object from Locale::Maketext for a user different to the current session user.

- Backport 1.221.

- Args to loncommon::getlangs() were reversed.

- Backport 1.220.

- Bug 5857.
- Retrieve senderemail addresses from stored addresses in the order: 'permanentemail','critnotification','notification'. Use the first non-blank entry found.
- Check for $env{'form.can_reply'}  and  $env{'form.reply_to_addr'} when setting 'From:' and 'Reply-to' addresses in e-mail header.
- Set $replytoaddress to 'do-not-reply@'.$hostname if can_reply is 'N'.

- Backport 1.219.

- Cache a user's language preference. Used when creating e-mail notifications to send to a user.

- Backport 1.218.

- Check if homeserver uses https protocol, and provide appropriate URL in notifiction e-mail.
- Remove machine specification with http or https from baseurl and attahmenturl when packaging message.

- Backport 1.216.

e-mail is generated, not message

- Moving &preferred_languages() and &get_genlanguages() from loncommon.pm to lonlocal.pm, so they cn be available in cases where lonlocal.pm is used but loncommon.pm is not.

Added/Converted POD comments

- New <group></group> entry in packaged message to store group for group broadcasts.
- Email to permanentemail, only send if notification has not also been sent to same address.
- process_sent_mail() takes extra argument - recipid, so it can be passed to &packagemsg() for access to recipients by type (To, Cc, Bcc, Broadcast etc.).

- Bug 5786. Language used for text wrapped around body of message in notification e-mail should correspond to language preference of recipient determined in order: user pref, domain default), unless message is sent in course content, and a language is defined for course (use course langage in this case).
lonmsg.pm - &user_lang() used to get a language hadler for recipient.
          - Translation of text uses &mt_user() instead of &mt().

lonlocal.pm - &mt_user() added (exported). Calls maketext, but takes language handler as firrst arg.

loncommon.pm - &get_genlanguages() added - functionality to turn en-ca into en-ca,en and also to remove duplicates from @preferred_languages - moved from &preferred_languages() to a separate subroutine so it is more widely available.

Bugs: 3706 5677 5678.
- Single attachment allowed in messages (whether sent via COM or FDBK).
- Warning displayed if uploaded file exceeded maximum size allowed for attachments (and was therefors discarded).
- Attachments included in forwarded messages (from both single message display, or from forward of multiple messages from folder display).
- Max size definition moved to a single location in lonmsgdisplay.pm and lonfeedback.pm
- Some appearance changes in modify_attachments display (for attachements associated with discussion posts).
- Subdirectory for storage of attachment file in sender's userfiles directory changed from feedback to feedback/$now (i.e., current UNIX time) to lessen likelihood of file being overwritten by a sequent message with an attachment of the same name.

Change arguments accepted by lonnet::appenv() to:
1. reference to hash
2. (optional) reference to array.

Change all instances where lonnet::appenv() is called to replace first argument passed (originally a hash) to a reference to the hash.

- Modify lonnet.pm rev 1.35 code intended to prevent "dangerous" modifications of the environment, so it will actually do this.
- Allow these modifications to be made in certain instances:
   DC switching to adhoc role
   CC switching to a different course role in the current course.
   user self-enrolling as a student

- switch from calling critical directly to using cput

- internationalize

Bug 5056.  E-mail notification excludes excerpt of if a non-critical message is received during a COM blocking period.

Substitution of HTML tags needs to occur one line earlier.

Bugs 4647 and 5164
- User can set option for HTML tags to be included in message excerpt sent in notification emails. Default is strip HTML.
- Interface changed for setting of notification email addresses - now set each address to receive all, critical only or non-critical only.
- javascript checks notification e-mail address to see if it is a valid format.
- Internationalization was missing in some places.

Include a subject line in sent mail when there is just a single recipient.

bug 5168.
- users with srm priv can designate a reply-to address (which may not be the same as the sender's) for messages
- users with srm priv can specify that a specific message should be eligible fo replies.
- some modification of button display for critical messages for consistency with this.
- Reply link elininated in essage display functions if message is not allowed relies
- Critical message receipts go to reply to address (if different from original sender).

- style police
- html police

Bug 4614
- Messages composed using COM screen can now specify either Cc or Bcc for the additional recipients textbox.
- recipients written as key = frozenhash to nohist_emailrecip db file in sender's space
- key stored in packaged message
- display of message includes Cc list
- when replying to message can use checkboxes for Ccs and/or "check all", "uncheck all" buttons to include Ccs in reply.
- Does not currently apply to users added via checkboxes in broadcast mode or specified in file in upload mode.

- Bug 3946. Messages sent with multiple recipients now included only once in "Sent Mail" folder, except when mode is "upload".

lonmsgdisplay.pm
- List of recipients included in message in "Sent Mail" folder for additional cases besides Critical and Broadcast.
- If a recipient has forwarding enabled, sender's record of recipients shows original recipient instead of the user account(s) to which it was forwarded.

lonmsg.pm
- Determination of course context of message moved to subroutine.
- User/domain of sent mail folder can be specified when storing sent mail.

- Bug 4551.  Users with dff privilege in a course can add a copy of a message to a User's Record (category Archive).
- Wording change for forwarding screen

- Making perldoc work in lonmsg.pm
- Moving some documentation from lonmsgdisplay.pm to lonmsg.pm
- Fix for critical messages sent to users with multiple forwards (lonmsgdisplay.pm rev 1.12) also needed to be applied for broadcast messages.
- Check for $specialmsg_status{$recipient} modified so sent mail will be stored in case of multiple forwards for critical or broadcast mail.
- Include sanity checking for ARRAY refs.

- starting work on moving to distributed DNS, eliminate usage of the lonnet hostanme global

- making error messages get sent again for print failures (BUG#2231)
  - moving decide_receivers to lonmsg for easier inclusion/use by
    cgi scripts

Got quite a lot of ISEs because of incomplete feedback address in course
environment - crash in line 573. That line should not be reached if there
is no valid recipient.

Getting the interpolation in &mt() straight.

Bug 5098
And some internationalization.

Bug 5116.  Also check for return status of 'ok' or 'con_delayed' to store sent mail or send a message to user's User Notes.

msgid to include escaped symb.

Include symb in msgid when sending feedback about a resource.

Include an error bit in msgid (1 if message is a "bomb" caused by an error when rendering a problem.

Include symb and resource title in messsage (<symb> and <resource_title> tags) when feedback is about a resource.

Navigate Contents page now checks for unread feedback or errors using symb instead of just resource url.

Navigate Contents page checks for resource context based on symb in msgid instead of in [url] included in message subject.  Backwards compatibility with old-style messages retained.

Subject for feedback messages about resources appends message title instead of url inside [].

Title on feedback page now avoids leaking unencrypted file name in cases where no title was assigned to a resource with hidden url.

When displaying feedback messages about a resource in a course, "Refers to" link displayed when viewer has corresponding course role selected includes symb in the link. Link text is now resource title.

"Refers to" link points to unencrypted resource url if feedback message is viewed under role other than original course, only if user has bre privilege for the resource, otherwise "Refers to" link is not displayed.

lonfeedback -- Some replacement of decode_symb() and &clutter() and &dewrapper() with &get_feedurl_and_clean_symb() for replies and edits of discussion posts.

lonfeedback -- More work on in ensuring hidden urls are encrypted or unencrypted as required.

bug 4745 sent message now stored for critical messages sent in on-course context.

Bug 4636.  Mail folders can now be deleted (if empty) or renamed.

email_status_X.db and nohist_email_X.db files used to store e-mail in different folders now have X set to a unique number (instead of the folder name) for newly created or renamed folders. Legacy data still supported.

Bug #4072: fudge "from" field

- lots of \w -> probper regexp replacements

Bug #4072: Add a reply-to field to notification messages.

- backport lonmsg.pm 1.185 lonmsgdisplay 1.31

- braak at the notes adding lines, for use elsewhere

&Apache::lonnet::unescape -> &unescape
&Apache::lonnet::escape -> &escape

- sending a critical message with a msgforward address set would cause messages to not get added to the users sent message folder

- backport 1.9 of lonmsgdisplay

- more properly hadle the bmsgiid subject (it is normally enocded then escaped, take care of undoing tat in the unpackmsgid now)

- adding machine that cuased the error to be added to the error message

- moving most of lonmsg into a new lonmsgdisplay module
- lonmsg now only contains the actual functions to send messages in the system and can now be ewasily used by now mod_pelr routines

- elminating a multi end_page occurance

- more start_paging, added the ddomain attribute to start_pagee

- page genearizlzation

- reeanbling RSS feed

- BUG#4699 - checking section 1 was also checking 10 11 etc

Eliminate duplication of message storage for broadcast and critical messages.  Message content now stored in a single record in nohist_email.db for each broadcast or critical mailing. Display of critical and broadcast messages in &disfacetoface() for individual students retrieves this message (based on msgid).

- stop adding citations to the sent message version
- don't display citations in the not a user.adv case

- err actaully correcting the style

	Style stuff.

- fix up the perm hadnling in messaging

- perf improvement, get all of the statuses we'll need in one ::get() rather than iterated gets

- BUG$4542, older messages didn't have a recuser,

When unpacking msgids, explicitly set $fromcid to '' for pre-1.140 msgids where processid is in position used for course context in 1.140 and later.  Speed up messages display by storing descriptions in a hash to avoid repeated lonnet::coursedescription() calls.

Speed up message display. Do not retrieve course description if $fromcid is '' (no course) or only digits (i.e., process id in old style msgid - rev 1.140 and earlier).

Be sure to set %status equal to the hash retrieved by lonnet::get();

Use lonmsg::unpackmsgid() to split msgid for sentmail.  remove some unnecessary unescaping from lond.

- typo

Use packagemsg() and unpackagemsg() routines in lonmsg.pm to store and retrieve DC mail, and eliminate packagmail() and unpackagemail() routines  (no longer needed) from lonnotify.pm.

- missed an RSS box

Acknowledgement of receipt e-mails now receive course context appropriate for course from which original critical message was sent.

Bug 3999.  Critical and Broadcast messages now handled similarly.  For each recipient a copy of the message is stored in the nohist_email db for the course, with subject Critical. [$user:$domain] or Broadcast. [$user:$domain] to allow for straightfoward construction of the Record page for each user in a course. A separate copy (with a different subjeca)t is stored in the nohist_email_sent db file, to alllow a more meaningful isubject in the sent mail folder display. The timestamp, courseID, and process ID parts of the msgIDs are common to the messages in the two db files.  Also double escaping of message content is no$w used for both Critical and Broadcast so User records will display subject and message in both cases.  Legacy broadcast messages still supported through use of Broadcast. as identifier of new style (compared with Broadcast for old style.

Bug 4454. Replies to COM messages that were originally sent in course context now retain the same course context even if replied to in a different context.

Bug 4333. Include course context in display of folder contents in COM. Can sort by course (description, NOT courseID).  Currently implemented for all folders. Could be restricted to just INBOX, as per bug request.

Bug 3946.  Only a single copy of a broadcast message is stored in the "Sent" Folder.  To: line now continues list of (multiple) recipients.

Implementing Course Journals

Having fun with RSS. Option when doing Broadcasts and Calendar Announcements

Next steps:
* dynamically generated, showing due dates, etc

Long term:
* podcasts
* blogs

Bug #3495: preview on mail

Back in town.

Only have grds and pprm links show if looking at resource message
Sorry for the syntax-bug submit

- reverting 1.149 not sure what's goal was and it made the server not start

Only show action links if info is available

Bug #2986: purge deleted messages immediately

Use existing functions

Direct links to track student, parameters, and grades from message, if
context is clear.

- more bug#2667 fix up all of the other paces doing the blind \n to <br /> tranlation

- bug #3715 html editor for COM style messages

- if in a sec only boradcast to that sec

COM: still forgetful about which folder I was in
GRDS: Added "Grading" to the Validate Button (confused me at first)
GRDS: Able to override the don't-reset-correct safeguard with checkbox

Adding course_id to msgid for LON-CAPA COM messages (uses course context of sender). Fixes to display of ungraded items, and change in parameters extracted from msgid to filter for course messages.

- ENV -> env

- jsMath wroks in COM

- better english for these buttons

- Fixes the WYSIWIG problems, the problem is since we are setting the encoding to UTF-8 on the homewor page, adn we _aren't_ setting it on the feed back generation page, when the WYSIWIG editor generates a   it becoms what ever rthe local encoding that was randomlypicked represents as a non-breaking space. Which muck things up to hell and back. So the fix is set the encoding to UTF-8 everywhere.

- xhtml cleanups
- stuck in a bunch of <label>s

- some of the </a> were in really weird places

- showing the list of students in has order doesn't seem like a good idea, how about name order instead

Saving my work on permanent email address copies - not yet tested.

Saving my work towards aspects of bugs 1290 and 3442.

* Display Folder from viewing a new message goes back to
  New Messages instead of INBOX.
* Notification email includes message text

- use <pre> rather than <blockquote> this finishes fixing 3507

Bug 3786:&discourse:Added '_' to the deliminator for student names so you
can actually send a message to students.

Bug #3771: not loosing status anymore when moving messages between folders.

Toward Bug #3771 - unclear why I had thought that email_status uses unescaped
msgids.

Bug #3772: Remember position in folder and number of displayed messages when
moving or deleting checked messages.

Bug #3668: "Back to Folder" goes back to correct position in folder
Bug #3709: additional safeguards against ISE on undefined first message number
Bug #3710: should not forget #messages anymore

Bug #3709: ISE on PREVIOUS

I could not get the ISE on line 956 to happen by using PREVIOUS, in spite of
doing lots of PREVIOUS, NEXT, FIRST, LAST - but I could make it happen with
an empty folder.

Detecting empty folders now. Will keep on trying.

Real fix for bug#3507.
Replacement of \n with <br> was breaking links where textarea had split
the <a from the href=" because it needed to break the line.  This is a radical
general change to satisfy a specific case.  If the replacement functionality is
desirable it might be better to parse the message and only make substitutions
where \n occurs outside HTML tags.

Fix bug #3507. Might want make this more general - e.g., just check for &Apache::lonnet::allowed('srm',$ENV{'request.course.id'}), so that all messages (not just crit messages) sent within course context avoid translation of HTML tags to > etc., if user has srm privilege.

Fixing Reply and Forward for messages in folders.

Storing my work on saved comments and smaller menu.

Don't forget the sorting

Make menu nicer for people than for Perl.

Wire up scrolling menu.

- er typo

Saving my work: cleanup

Further work on cleaning up menus

Saving my work: trying to consolidate INBOX and Critical Messages into folder
idea.

Trying to clean up and consolidate before attacking folders again.

Show current state of resource if available. Also on Reply screen.

Duh.  Fix stuff from the prior commit.

use loncoursedata::get_classlist instead of doing the classlist dump ourselves.

Just saving work from during the trip

Saving my work:
* orange on green did not look great for help link (something still fishy
with use of &mt there)
* moving between folders works now, TRASH is still not quite working right

Quickly saving my work on folders in COM, Trash.
Also fixed bodytag "emfo" - BTW, bodytag does its own &mt-call.

- replied status was no longer being set (BUG#3425)

Added support for "Gradebook: Ability to add text to rep".
lonmsg.pm: Added &retrieve_instructor_comments
lonstatistics.pm: Added 'comments' field to student data selector.
lonaboutme.pm: Added anchor for comments section.
lonstudentassessment: Added output for "comments" column:
    html: link is included
    excel & csv: text is included, although excel has limits.

- stupid %ENV problem again

Broadcast messages now appear in display of user notes, record of face to face discussions, critical messages, and broadcast messages for each student.  Assigned to me as item 7 in "1.2 Plans" - LON-CAPA dev list 2/10.  Partial solution to bug #2331.

Course Coordinators and Instructors can schedule blocking periods for student display of e-mail.
During a blocking period students will be unable to display LON-CAPA e-mail sent during the current
blocking period. If Course Coodinators or Instructors need to contact students during this time, they
should send a critical message (these will still be viewable). Note: because COM functions can be
used outside a course context, blocking prevents display of all messages sent to the student during the
blocking period (including other courses).

- feedback message attachements now work

- finishuploadfile works better with a directory specfied
- bb posts, and emails no longer use token wrapper

Added 'tex' target neccessary to work with lonaboutme.pm.

Bug 2147 (email bug)

- trying to be UTF8 transparent

Improve workflow after sending messages, fix "Cancel"
Start of experiment with lonindexer.pm to display pages and sequences

Better screen output.

&comprep was removed, need to call &compout

* Bug #2500: Reply now has "additional recipient"
* Removal of redundant code

- backport 1.86

XMLify

New headers and breadcrumbs

Adding &mt() to crumbs display.
Saving my work on lonmsg.

Saving my work (some headers lost):
* trying to slowly clean up
* use breadcrumbs
* internationalize

Re-tabinate.

Bug #2784: message from uploaded file was not sent.
For 1.1.2 and s10

Bug #2604: add course name to feedback
Bug #2676: add section to feedback

Bug #2666: Explanation of what "Confirm Receipt" does.

Bug #2528: Some words of confirmation after successfully mailing a message.

Bug #2640: Add subject to receipt subject.

- Fixes BUG#2499, emails with large <m> where getting shoved onto one line because
   - we use <pre>
   - we strip out extra returns before processing

- I tired these changes out with a variety of messages both with and without <m> sections, and post discussion, looks good to me. let me know if you see anything else that looks ugly

Rewinding to 1.78 - 1.79 not a good idea.

Just saving my work.

FAQ and Bugs

Bug #2444: error messages can be displayed from Construction Space

Bombs in "Remarks" field on Roles screen for author roles with bombs.

Bug #2444: show bombs in construction space.

Work on Bug #2444: retrieval of bombs possible again.

Bug #2444: saving my work, does not work yet.

Rollback to 1.70 in order to really fix the bug.

- Fixes, BUG#2419, (author's not seeing error messages), but only by converting to error messages, delaying real fix to 1.2 (BUG#2444)

Bug #2298: now whines louder if it cannot deliver a message.

Bug #2171: Cancel button on send and send reply

Trying to address Bug #2197: do not print to mail handle if it could not be
opened.

I was going to work on something else ... anyway, more i18n.

Internationalization, and in particular, lonlocal:locallocaltime and
Japanese dates.

Bug #2031: Next and Previous buttons when viewing messages
Jon Kitzman
combined with functionality bug #788 (Kitzman) to keep sorting order
when doing "Next" and "Previous"
We never had a routine to get the keys of a namespace in lonnet.pm

Better fix of Bug #788.

- retabination

Bug #788
Sorting mail
Jon Kitzman
- now clickable table headers to sort in different ways.

Bug #1766: be able to broadcast messages to roles other than student

Bug #2017: don't leave user hanging when no critical messages left

I screwed up. Please do not hurt me.

Documentation updates

Have very limited connectivity, need to do big commit while online.
Will call this JULYone in Bugzilla. Fixes several small bugs.

- fixes BUG#1809, call clear_out_html for chat and internal messages

Add Latex cheatsheet.

Direct notification email directly to login screen.

Sends notification emails.

Attachments displayed in normal messages.

Attachments in discussions work, but not yet in other messages.

-removing baseurl testing logsthises

- bacport of 1.48 (fully encode entities in messages)

-Fixes BUG#1255, sets the base url of email messages that contain forwars of a problems view, thus relative links will work.

- removing old code

-adding answers to feddback messages

VS: ----------------------------------------------------------------------

Finished "Select Student" screen.
Incorporation of the "Select Student" screen in messages and parmset.
Clean-up for "Face-To-Face" functionality.

Bug #274 Face-to-face discussion (and critical messages) recorded.

Toward bug #274 - recording of face-to-face discussion (look-up of all student
communication)

Bug #1007 - Delete link on View Message screen.

Bug #782

Actually check for mail

Mark if new email came in

- should be better about silly messages (I keep get this message listed from 1969 that is ne no matter what I do with sending myself messages our trying to delete. With this it at least stops showing up.)

Bug 707
Renamed old sending function xxx_raw, and made same-name new functions that
look at user's environment "msgforward".

Convenience links to about-me page, syllabus, and email.

Email can also contain LaTeX

Checking commit of scripts to make sure I don't get too far out of sync. Added help
links to the message sender describing 'critical message', and several help links in
the course parameters screen. (These will not work until some .tex files are checked
in.)

Bug 509 - shows subject of critical messages in addition to sender and date.

Get rid of spurious error-message-email if user did never have email.

Changed to use loncommon::get_unprocessed_cgi().  Bug 90.

Minor cleanups, call to &Apache::loncommon::select_dom_form.

Handle case of not having critical messages cleanly.

Check/Uncheck All messages to be deleted.

map to foreach

Full subject (so URLs get included)

- Finished GPLing the code

Bulk delete

Receipt of critical is optional.

Hard line wraps.

Mail from uploaded file

Did not log msgid

Additional recipients

Send to group/section

Broadcast Messages

Start on Broadcast

Can forward and send message to individual user

More functions

Mark as deleted.

Reply to critical message

Receipt of critical messages

More email functions

- changed the put() and cput() calls
- updated all locations that used them
- changed lonnet::modifyuser() to use new get and put.

- modified eget and del in lonnet to take optiona domain and user args
- modified lonmsg usr_crit_received to use new semantics
- retrofitted devalidate to use the new more powerful del

- modified get to now use an array reference, and to accept to optional
  arguments $udom and $uname to specify who we are talking about.
- modified all current uses of get() to pass array references
- fixed bookmarks

Feedback works, kind of

Trying to implement mail reader

Logs critical messages

Bug Fixes and Received Function

Bug fixes

Adding functionality

Module to deal with messaging and error messages