CVS log for loncom/interface/lonmodifycourse.pm

Up to [LON-CAPA] / loncom / interface

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

- Report OK when the PerlCleanupHandler has successfully finished execution.

- For 2.11.4 (modified)
  Reinstate line removed in 1.79.2.9.2.6

- For 2.11.4 (modified)
  Include changes in 1.104 (part), 1.105, 1.106

- Remove trailing whitespace.

- Available editors in Course Authoring Space, or when editing an html file
  created in a course folder using the Course Editor is a domain default,
  which can be overridden in specific course(s) by a Domain Coordinator.

- Availability of "Course Authoring Space" set in Course Community defaults
  in domain (by container type), can be overridden for specific course(s).
- Add some help files for Course Editor and View/modify a course/community
- Add icons for use of Course Authoring Space.

- For 2.11.4 (modified)
  Include changes in 1.102 and 1.103.

- Remove trailing whitespace. No code changes.

- For 2.11.4 (modified)
  Include changes 1.101

- "View or modify a course or community" item for Domain Coordinators.
  - "Actions" footer included on all pages after selecting a view/modification
    action for a specific course/community from "View/Modify Course Settings'
  - Course type included in header line above icons/links for actions.
  - Course title moved to <h4> header line on actions pages.
  - Explanation of actions moved to help file accessed via question mark on
    right side of breadcrumbs.
  - Satisfy w3c validation

- Domain configuration for default cumulative quota for all group portfolio
  spaces in course can be overridden for a specific course by a Domain
  Coordinator.

- For 2.11.4 (modified)
  Include changes in 1.93, 1.99, 1.100

- Typo in rev. 1.99

- Bug 6754
  - Domain configuration to set defaults for which course types can use:
    (a) External Tools defined/configured in domain, and (b) External Tools
    defined/configured in course.
  - Domain defaults can be overridden by DC for specific course(s) via:
    Main Menu > "View or modify a course or community".

- For 2.11.4 (modified)
  Include changes in 1.98

- Fix typo

- For 2.11.4 (modified)
  Include changes in 1.97

- Bug 6907
  - Domain default to determine whether LTI launch of deep-linked URL requires
    student to authenticate; can be overridden for specific course(s).
  - If domain config permits it, link protection setting in a course can
    specify whether to accept username included in LTI payload, and action
    to take if username is not for an enrolled student.
  - Second arg passed to ltiauth::invalid_request() is text string stating why
    LTI launch was invalid.

- For 2.11
  Backport 1.96

- Support customization in localenroll.pm to eliminate ambiguity in extraction
  of institutional section from institutional course section for automated
  enrollment.

- For 2.11
  Backport 1.94

- For 2.11
  Backport 1.95

- Devalidate cache for self-cataloging of courses based on institutional
  code for institution's nodes, if creating/modifying an "official" course.

- Support domain configuration which allows a Course Owner to change a
  student's password, if:
 (a) same domain is used by owner, course, and student
 (b) student has no active or future roles besides student role in courses
     owned by the course owner making the change
 (c) course container is not Community or Placement Test
 (d) owner is course cordinator in the course
 (e) setting to disable this action has not been set for the specific course

- Bug 6754 LON-CAPA as LTI Provider
  - Creation of new LON-CAPA user account available from an LTI Consumer.
  - Creation of new LON-CAPA course available from an LTI Consumer.
  - Self-enrollment in a LON-CAPA course available from an LTI Consumer.

- For 2.11
  - Backport 1.92

- Hide role selection text used in modal window until needed.

- For 2.11
  - Backport 1.90, 1.91

- white space change.

Ad hoc roles for helpdesk personnel.
  - Entering a course with one of the available ad hoc roles via
    Main Menu > View a course or community (Domain helpdesk role)
    uses modal pop-up to select role, after choosing course, if more
    than one ad hoc role is available to current user.

- For 2.11
  - Backport 1.87, 1.88, 1.89

- Remove duplicate comma.
- Add white space before tag closure
- Remove superfluous quotes
- Use permission-based link text for link to configure or view setting for
  post-submission behavior of submit button

- Domain Helpdesk (with rar priv) can view (but not edit) course settings
  set/modified by a Domain Coordinator.

- Remove trailing white space.

- For 2.11
  - Backport 1.86

- Domain configuration (Support Settings) allows custom course role(s) to be
  defined which will be available for domain helpdesk staff to select as
  ad hoc role(s) via "View a course or community" in main menu.

- For 2.11
  - Backport 1.85

- Lifetime of temporary md5_* tables (used to speed up access to student
  performance data, e.g., for Chart or Statistics), can be set as domain
  default (based on course type), and be overridden by a DC in a course.

- For 2.11
  - Backport 1.84

- Bug 5464
  Can set failsafe value in domain (or override in Auto-enollment course
  configuration to avoid unwanted dropping of students when no data are
  retrieved for a particular inststitutional section.

- Bug 6808. New course container -- "Placement" for Placement Tests.
  - Placement Tests can have an instituional course code, so that
    auto-enrollment can be supported.

- Remove debug line.

- Bug 6808. New course container -- "Placement" for Placement Tests.

- Bug 6808. New course container -- "Placement" for Placement Tests.
  Work in progress.

- Fix typo.

fixed bug 6782, and escaped most localized messages used in Javascript blocks to make sure bugs like that do not happen again

- Modify args passed to lonpickcourse::display_matched_courses (changed in
  rev. 1.116).

Typo.

- Additional Course/Community defaults for JSME Molecule Editor and Math
  Previewer (can be overridden in a course by CC), and for disabling submit
  button after student submission, with/without timeout, setby course container
  type (can be overriden for specific course by a DC).

Making drop-down selections persistent after search

- Display of self-enrollment settings, and saving of changes takes into
  account domain configuration (or course-specific configuration) which
  determine which self-enrollment settings in a course can be set by a CC.

- Where a course is set for configuration of one or more self-enrollment
  settings by a DC instead of by course personnel, DC's modify course
  menu includes access to self-enrollment configuration.
- Self-enrollment supports option to queue request, pending validation
  by external site (e.g., separate payment site for e-commerce).
- &get_selfenroll_titles() removed from loncreateuser.pm (use subroutine
  in lonuserutils.pm instead).

- Modify &check_course() to return additional item (ref to %coursehash),
  and eliminate passing of unused request object.

- &search_courses() and &js_changer() moved from lonpickcourse.pm to
  loncommon.pm to facilitate reuse.

- Moving &build_filters(), &timebased_select_form, &js_changer()
  from lonpickcourse.pm to loncommon.pm to facilitate re-use.

- Institutional code is stored as internal.coursecode in a course's
  environment.db

Internationalization: Proper translation string compilation

Improved and consistent navigation and layout by using actionbox

- New course request type -- textbook.
  - Can assign rights to request this type of course.
  - "textbook" can have different uploaded content quota, default credits etc.
    from official or unofficial course types.
  - Domain Coordinators can specify subject, tile, authors(s) and upload an
    image of the textbook cover.
- Work-in-progress.

- Consistent wording.

- Wording changes.

Exclude HTML tags from translation phrases.
Update corresponding German translation file phrases and translations.

- Disk quotas for content uploaded directly to a course (userfiles/docs/ and
   userfiles/supplemental).
  - Default is 500 MB, unless set at domain level, or individual course level.
  - Domain defaults set by DC  for: official courses, unofficial courses and
    communities set via "Course/Community defaults" in "Set domain configuration".
  - Quota for individual courses/communities set  by DC via:
      "View or modify a course or community"
  - &excess_filesize_authorspace() renamed as excess_filesize_warning() to
    support quota checking in course context, as well as in author context.

- DC can modify course default credits value if domain configuration set
  to show credits for official or unofficial courses.

- scope

- Include co-owners as argument passed to lonnet::auto_new_course() whenever sections, crosslistings or course owner change in an "official" course.

- Customization for GCI_3.
  - Typo in 1.57.2.1.

- Customization for GCI_3.
  - Changes in 1.56 excluded from version for GCI_3.

- DC can set number of responders needed when CC displays anonymous survey
  submissions.

- Bug 6333.
  Ensure that form element names for select boxes for institutional codes
  are always named Year, Semester, Department, Number, for consistency with
  names used in javascript to update select box items dynamically.
  - Names of categories in domain from localenroll::instcode_format():
    codetitles array can differ from these form element names.

- Update/New icons for DC sub menus:
(Currently same icons for courses and communities)
    - Icons for sub menu "Course and community creation"
      (icon for "Display official course requests" still missing due to unclear functionality)
    - Icons for sub menu after having selected a course with "View or modify a course or community"

- Use new "course/community configuration" icon for course/community settings as DC and for course/community configuration as CC

Credits: Juliane Wenzel

Bug 6273 - Consistent menus (use generate_menu)
- View/Modify settings for a course (DC settings)
- Status of domain servers
- Use category icon for each menu item (planned to be replaced with individual version)

Included changes:
- lonmodifycourse.pm:
    - XHTML - escape "&" in Link URLs
    - XHTML/context - enclose form elements in form
    - Localization: optimized &mt() usage in headline
- domainstatus.pm:
    - Removed now unnecessary divs to float header and footer
- de.pm:
    - Added translations

- Consistent wording.

- &Apache::lonnet:: prefix for calls to appenv() and put() subroutines.

- When auto-assignment of co-owners is enabled for official courses:
 - Use lonnet:: auto_validate_instcode() to check active CC is official
   course personnel before assigning co-owner role when
   (a) modifying institutional code, or (b) changing course owner.

- Co-owners.
  - New routine: &update_coowners() will adjust co-owners new owner used to
    be a co-owner.
    - If this is an official course, and auto-assignment of co-owners is
      enabled, co-owner status will be updated for any active CCs
      who are validated as official course personnel.

- Accommodate Communities
  - Top Level Category: "Communities" defined in system
    (Domain control to hide/unhide).
    - Can only assign communities to this category (and its subcategories).
- Domain Coordinator definition of new top level categories:
   instcode and communities are reserved for system use
   (may not be used for names of additional top level categories).

- Accommodate Communities.
- More concise wording
- Links to course's 'Automated Enrollment Manager' via role switcher.
- For Courses use pick_box() in interface for modification of institutional code,
    default authentication and owner.
- Eliminate unused code.

Consistent wording "Course Catalog" -> "Course/Community Catalog"

- When used in course request or course creation context, courses display replaces select button with text warning for courses for which owner lacks cloning rights.
- In those contexts can filter based on cloneability.
- Hierarchy of linked select boxes used for institutional codes (with official/unofficial radio button selectors) now used for "Modify Course" function available to Domain Coordinator.
- Eliminate "command" form element added in lonpickcourse.pm rev 1.84. Revert to check for g'osearch' form element to trigger search.
-  No search if page updated by onchange event for Type or Domain
   (as detected based on non-null value of updater form element).

- Corrected nested HTML tags structure (<form> / <p>)
- Corrected unbalanced tags (added </p> to menu; removed </p> from kerberos warning)
- Replaced hardcoded font color by appropriate warning style (kerberos warning)
- Removed unused color variables
- Optimized &mt usage
- Added some line breaks for better code readability

- Restoring $caller as the last argument (now eighth) in lonpickcourse::build_filters().
  (removed in rev 1.78).
- Modifying calls to lonpickcourse.pm routines: build_filters() and display_matched_courses() in lonmodifycourse.pm, to take account of changed args.

consistent wording:
changed "group files" to "group portfolio"
changed "file repository" to "group portfolio"
changed "group portfolio files" to "group portfolio"

short button names, german translation of "token".

- If $component contained course title
  &lonhtmlcommon::breadcrumbs() would generate ISE is title included [].
  - Do &mt() call before passing $component to lonhtmlcommon::breadcrumbs()
  - Additional argument (no_mt) in lonhtmlcommon::breadcrumbs().
- Append course title to "Current automated enrollment settings for " heading.

Typo

- Wording changes: actions are actually available after course search, not after course selection.
- Punctuation.
- Trim an extra line feed.

- Ability to exclude a course from the course catalog.
- Ability to categorize a course from categories defined for the course's domain.

Bug 4323. Ability for DC to enter a course as a Course Coordinator from the Main Menu, in case the user has only one role (DC), so roles screen is skipped on log-in.
- Some spaces added to text displayed for matched courses in course search.

Calls to routines from londropadd replaced with calls to the same routines in lonuserutils.

- Use lonnet::coursedescription() to extract cnum from cid instead of splitting on underscore.
- Formname for page to change DC-modifiable enrollment settings set to setparms, so
 breadcrumbs links work
- Fix scope in &check_course() so $cdesc is retrieved.

Bug 5191.  Decouple storage of last access time for a course in nohist_courseids.db from storage of course information.
- Retain backwards compatibility with legacy lonnet::courseiddump()

Other general change: uniform use of 'inst_code' as key in frozen hash instead of 'instcode'.

lond
&put_course_id_handler()
- last access stored in escaped lasttime:courseid key = value hash item
  if legacy lonnet::courseiddump() updates courseinfo.
&put_course_id_hash_handler()
- additional argument - $mode ('timeonly', 'notime' or 'all')
  - determines whether last access, or courseinfo (description,inst_code,owner,type) or both are stored
&dump_course_id_handler()
- documentation updated
- style (replacing unless)
- change location of unescaping when checking filters so double escaping no longer needed for courseinfo stored in frozen hash
- return courseinfo in hash form, if requested as hash, when record has yet to be converted to a frozen hash
&validate_class_access_handler()
- eliminate extra unescaping of $ownerlist

lonnet.pm
&flushcourselogs() includes 'timeonly' argument in call to courseidput().
&courseidput() - if in 'timeonly' mode only courseids are sent to lond in courseidputhash call.
&createcourse()
- courseinfo values in hash no longer escaped (freeze_hash handles this)
- call to flushcourselogs() replaced by call to courseidput()

lonmodifycourse.pm
- courseinfo values in hash no longer escaped
- call to courseidput() includes 'notime' as fourth arg.

coursecatalog.pm
- remove unneeded code, since entry for each course in %courses hash will now be a hash ref.

courseclassifier.pm
- remove unneeded code, since entry for each course in %courses hash will now be a hash ref.

lonpickcourse.pm
- Entry for each course in %courses hash should be a hash ref, when constructing for onlyown courses case.
- instcode to inst_code
- no need to unescape courseinfo value (thaw_hash in lonnet::courseiddump does it)

lonparmset.pm
- call to flushcourselogs() replaced by call to courseidput() after updating course description.
- appenv user's session with new course description.

Store data in nohist_courseids.db as a hash.  Maintain backwards compatibility with writes to the db from legacy lonnet.pm, and also reads from the db from legacy lonnet.pm.

Start towards inclusion of co-owners as well as a single owner, for the purposes of verifying access to official classlist data.

lond
- &put_course_id_hash_handler() added to handle storage of data in nohist_courseids.db as a frozen hash.
- &put_course_id_handler() will add to an existing hash if update is from a legacy lonnet which supplies : separated data.
- &dump_course_id_handler() $as_hash flag extracted from tail. If set, data are returned as a hash.
- validate_class_access_handler() will extract comma-separated owner and co-owners from ownerlist passed to it, and send array ref to localenroll::check_section() to determine if at least one on the owners is authorized to view classlist list data for the institutional section.

lonnet.pm
- storage of hash via call to courseidputhash in lond.  If command is unrecognized on the lond side, subset of the data sent as : separated values
- retrieval of data from nohist_courseids.db - additional $as_hash argument included in courseiddump call to indicate that data should be returned as a frozen hash.
- auto_validate_class_sec() - if third argument is an array ref, then an owner list is built from the supplied owners and co-owners.
- lasttime set on lonnet side (FIXME) - to be changed to lond side.

lomodifycourse.pm
- updated course settings sent as a ref to a hash to lonnet::courseidput()
- include LC_error in span for error messages
- eliminate duplicate error message
- display error if storage of changes fails
- form name corrected to cmod.

coursecatalog.pm
- eliminate unused arg from call to courseclassifier::retrieve_instcodes
- display owner and co-owners (sort by lastname)
- institutional section access for multiple owners

courseclassifier.pm
- course codes from course data if hash, or if : separated string.

lonpickcourse.pm
- course info extracted from hash
- owner info displayed in a list if owner and co-owners exist.

Only allow selection of default authentication method used for auto-enrollment in a course from types available to Domain Coordinator (set in domain config).

- eliminating the domain hash globals in favor of functional access

Prevent stripping of trailing null fields in split() by setting limit to -1.

Undesired stripping of trailing null fields had caused extra : separators to be written in values in nohist_courseids.db between type entry and final entry (last access), in some cases where there was no course owner.  Set numcurrent to a maximum to eliminate the extra items the next time put_course_id_handler() runs to update the entry for a particular courseID.

Include type in courseidput when modify_course() results in change in course owner or institutional code.

Typo (sectionums should be sectionnums).

Set default $type to 'Course', in case the version of lonpickcourse.pm being used has no Course/Non-standard course selector.

Selection of course to be modified now uses routines from lonpickcourse.

Added ability to set course groups portfolio files quota.

Separated display of all current auto-enrollment settings for course from page for setting the four DC-only modifiable settings.

Replaced existing breadcrumbs with standard LON-CAPA breadcrumbs.

Use standard LON-CAPA data table functions.

Some reorganization into subroutines.

- xhtml

Bug 4813.  Retain backwards compatibility with legacy courses in which only a course owner's username is stored in environment.internal.courseowner, and in the course's record in nohist_courseids.db.  New courses will include username:domain (escaped) in nohist_courseids.db.  MCRS can be used to update legacy course owner information (just click "Modify Settings" button, it will be necessary to select the correct course owner where the course domain is different from the owner's domain.

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

-start_page
-eliminate calls to bodytag in favor of start_page
- elimintate call to xmlbegin

- page genearizlzation

- add a space so long lists wrap

- eliminate some overuse of $1

- adding version string and 'purpose of file' line

Moved check_user_status() from Enrollment.pm to loncommon.pm.  Now returns: active, previous or future (instead of ok or expired)

Add extra argument to pass to courseiddump for LON-CAPA courseID (as required by /lonnet/perl/lonnet.pm rev. 1.622).

Bug 3912.  DCs can use courseID (e.g., 257472759ae4061msul1) as a filter when using pickcourse.

- ENV -> env

- <html> -> &Apache::lonxml::xmlbegin, thus valid doctypes are now getting output, Yeah! StandardsCmpliance

- backing out the encoding changes for now

- some xhtml cleanups
- one icon -> lonhttpd

Adding storage of course owner to nohist_courseids.db.  Pick course can permits use of institutional code and/or username of course owner as filters.

localenroll no longer used directly.  Domain to check is domain for DC role, not home domain of user.

Domain should be domain of current DC's role instead of user's home domain.

Bug #3212 Permit lower case Kerberos domain names.

Include institutional course code in nohist_courseids.db.
Format of entries in nohist_courseids.db now -
course_id = description:institutional code:last activity
both (keys and values are escaped).
lonnet::courseiddump now returns escaped values in hash (keys are unescaped),
to preserve the : separator between description and institutional code.

Calls to functions in localenroll.pm mediated by lonc/lond now include domain
in the set of passed parameters to support domain-specific variants in a single
localenroll.pm file housed on a multi-domain library server.

Automated enrollment now sets first access and last access dates for student
roles based on defaults set on course creation, and modifiable via a number
of mechanisms in the Enrollment Manager, and Automated Enrollment Manager.

The Automated Enrollment Manager now provides a utility to allow enrollment type
to be toggled between 'auto' and 'manual', so that students added via CSV file
or singly can become liable for automated update (dropping, section switching
etc.).

Fix bug #3062

Changes to support autoenroll calls from a remote server.

lond on the homeserver for the course(s) handles requests from the remote
server for institutional data (e.g., classlists, valid institutional courseIDs,
institutional section numbers for a course code, validation of course owners), by
calling the appropriate functions in the homeserver's localenroll.pm

All replies are made directly with the exception of fetch_enrollment_query, which
is shipped over to lonsql, in case retrieval of institutional classlists is a
protracted process.

lonsql on the homeserver for the course(s) calls localenroll::fetch_enrollment()
and writes XML files of enrollment data to /home/httpd/perl/tmp

Transfer of classlist data occurs later following an autoretrieve call from the
remote server. It is planned to generalize this function and add encryption to the transfer back to the client.

Autoenroll.pl called by cron on a library server, now only carries out updates for
courses in its domain, for which the library server is the course's homeserver. If a domain has multiple library servers Autoenroll.pl will need to be run on each library server.

Fix a bug and an indent

Eliminate senseless array stuffing in &check_course().  Correct indents.

Added course owner validation for cases where course owner is changed, but course code is unchanged.

Added sanity checking for valid LON-CAPA course ID.

Completed localization. Some changes to menu layout.

For use by domain coordinators to change course code, course owner, default authentication mechanism in any course in the domain.  All of these parameters are used by the automated enrollment process.