CHANGELOG.md

@@ -3,10 +3,6 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
-## [0.3.6]
-### Fixed
-- `--all` once again downloads all courses you're a member of
-
 ## [0.3.5]
 ### Added
 - `--pass-path` option to get the password from [pass](https://www.passwordstore.org/) (PR [#33] by [@Ma27])

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "KIT-ILIAS-downloader"
-version = "0.3.6"
+version = "0.3.5"
 authors = ["FliegendeWurst <2012gdwu@posteo.de>"]
 license = "GPL-3.0-or-later"
 edition = "2018"

src/cli.rs

@@ -174,10 +174,14 @@ pub fn ask_user_pass(opt: &Opt) -> Result<(String, String)> {
 				error!(e);
 				pass = rpassword::prompt_password("Password: ").context("password prompt")?;
 				should_store = true;
-			},
+			}
 		}
 	} else if let Some(pass_path) = &opt.pass_path {
-		let pw_out = Command::new("pass").arg("show").arg(pass_path).output().map_err(|x| {
+		let pw_out = Command::new("pass")
+			.arg("show")
+			.arg(pass_path)
+			.output()
+			.map_err(|x| {
 				if x.kind() == ErrorKind::NotFound {
 					Error::new(ErrorKind::NotFound, "pass not found in $PATH!")
 				} else {
@@ -189,19 +193,16 @@ pub fn ask_user_pass(opt: &Opt) -> Result<(String, String)> {
 				ErrorKind::Other,
 				format!(
 					"`pass` failed with non-zero exit code {}: {}",
-					pw_out.status.code().expect("Failed retrieving pass exit code!"),
+					pw_out.status.code()
-					String::from_utf8(pw_out.stderr).expect("Failed decoding stderr of pass!")
+						.expect("Failed retrieving pass exit code!"),
-				),
+					String::from_utf8(pw_out.stderr)
-			))?;
+						.expect("Failed decoding stderr of pass!")
+				)
+			))?
 		} else {
-			pass = String::from_utf8(pw_out.stdout)
+			pass = String::from_utf8(pw_out.stdout).map(|x| {
-				.map(|x| {
+				x.lines().next().map(|x| x.to_owned()).ok_or_else(|| anyhow!("empty pass(1) entry!"))
-					x.lines()
+			})?.expect("utf-8 decode of `pass(1)`-output failed");
-						.next()
-						.map(|x| x.to_owned())
-						.ok_or_else(|| anyhow!("empty pass(1) entry"))
-				})?
-				.expect("utf-8 decode of `pass(1)`-output failed");
 			should_store = false;
 		}
 	} else {

src/ilias.rs

@@ -28,10 +28,8 @@ static ALERT_DANGER: Lazy<Selector> = Lazy::new(|| Selector::parse("div.alert-da
 static IL_CONTENT_CONTAINER: Lazy<Selector> = Lazy::new(|| Selector::parse("#il_center_col").unwrap());
 static BLOCK_FAVORITES: Lazy<Selector> = Lazy::new(|| Selector::parse("#block_pditems_0").unwrap());
 static ITEM_PROP: Lazy<Selector> = Lazy::new(|| Selector::parse("span.il_ItemProperty").unwrap());
-static CONTAINER_ITEMS: Lazy<Selector> =
+static CONTAINER_ITEMS: Lazy<Selector> = Lazy::new(|| Selector::parse("div.il_ContainerListItem, .il-std-item").unwrap());
-	Lazy::new(|| Selector::parse("div.il_ContainerListItem, .il-std-item").unwrap());
+static CONTAINER_ITEM_TITLE: Lazy<Selector> = Lazy::new(|| Selector::parse("a.il_ContainerItemTitle, .il-item-title > a").unwrap());
-static CONTAINER_ITEM_TITLE: Lazy<Selector> =
-	Lazy::new(|| Selector::parse("a.il_ContainerItemTitle, .il-item-title > a").unwrap());
 
 pub struct ILIAS {
 	pub opt: Opt,
@@ -343,7 +341,7 @@ impl Object {
 			| Generic { name, .. } => &name,
 			Thread { url } => &url.thr_pk.as_ref().unwrap(),
 			Video { url } => &url.url,
-			Dashboard { url } => &url.url,
+			Dashboard { url } => &url.url
 		}
 	}
 

src/ilias/folder.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashSet, path::Path, sync::Arc};
+use std::{path::Path, sync::Arc, collections::HashSet};
 
 use anyhow::{Context, Result};
 
@@ -23,7 +23,9 @@ pub async fn download(path: &Path, ilias: Arc<ILIAS>, url: &URL) -> Result<()> {
 	let mut names = HashSet::new();
 	for item in content.0 {
 		let item = item?;
-		let item_name = file_escape(ilias.course_names.get(item.name()).map(|x| &**x).unwrap_or(item.name()));
+		let item_name = file_escape(
+			ilias.course_names.get(item.name()).map(|x| &**x).unwrap_or(item.name()),
+		);
 		if names.contains(&item_name) {
 			warning!(format => "folder {} contains duplicated folder {:?}", path.display(), item_name);
 		}

src/ilias/plugin_dispatch.rs

@@ -14,9 +14,7 @@ static LINKS: Lazy<Selector> = Lazy::new(|| Selector::parse("a").unwrap());
 static A_TARGET_BLANK: Lazy<Selector> = Lazy::new(|| Selector::parse(r#"a[target="_blank"]"#).unwrap());
 static VIDEO_ROWS: Lazy<Selector> = Lazy::new(|| Selector::parse(".ilTableOuter > div > table > tbody > tr").unwrap());
 static TABLE_CELLS: Lazy<Selector> = Lazy::new(|| Selector::parse("td").unwrap());
-static LIST_URL: Lazy<Regex> = Lazy::new(|| {
+static LIST_URL: Lazy<Regex> = Lazy::new(|| Regex::new("ilias\\.php\\?ref_id=\\d+&cmdClass=xocteventgui&cmdNode=.{9}&baseClass=ilObjPluginDispatchGUI.*&cmd=asyncGetTableGUI&cmdMode=asynch").unwrap());
-	Regex::new("ilias\\.php\\?ref_id=\\d+&cmdClass=xocteventgui&cmdNode=.{9}&baseClass=ilObjPluginDispatchGUI.*&cmd=asyncGetTableGUI&cmdMode=asynch").unwrap()
-});
 
 const NO_ENTRIES: &str = "Keine Einträge";
 

src/main.rs

@@ -17,6 +17,7 @@ use std::sync::Arc;
 use std::time::SystemTime;
 
 static ILIAS_URL: &str = "https://ilias.studium.kit.edu/";
+/// main personal desktop
 static DEFAULT_SYNC_URL: &str =
 	"https://ilias.studium.kit.edu/ilias.php?baseClass=ilDashboardGUI&cmd=jumpToMemberships";
 
@@ -160,10 +161,8 @@ async fn real_main(mut opt: Opt) -> Result<()> {
 	}
 
 	let sync_url = if ilias.opt.all {
-		format!(
+		// change on ILIAS update
-			"{}ilias.php?cmdClass=ilmembershipoverviewgui&baseClass=ilmembershipoverviewgui",
+		format!("{}ilias.php?cmdClass=ilmembershipoverviewgui&cmdNode=iy&baseClass=ilmembershipoverviewgui", ILIAS_URL)
-			ILIAS_URL
-		)
 	} else {
 		ilias.opt.sync_url.as_deref().unwrap_or(DEFAULT_SYNC_URL).to_owned()
 	};