Revert "Support anchors in SUMMARY (#1173)"

This reverts commit 2baed040c2.

src/book/book.rs

@@ -153,8 +153,6 @@ pub struct Chapter {
     pub sub_items: Vec<BookItem>,
     /// The chapter's location, relative to the `SUMMARY.md` file.
     pub path: PathBuf,
-    /// An optional anchor in the original link.
-    pub anchor: Option<String>,
     /// An ordered list of the names of each chapter above this one, in the hierarchy.
     pub parent_names: Vec<String>,
 }
@@ -245,7 +243,6 @@ fn load_chapter<P: AsRef<Path>>(
     let mut sub_item_parents = parent_names.clone();
     let mut ch = Chapter::new(&link.name, content, stripped, parent_names);
     ch.number = link.number.clone();
-    ch.anchor = link.anchor.clone();
 
     sub_item_parents.push(link.name.clone());
     let sub_items = link
@@ -323,7 +320,7 @@ And here is some \
             .write_all(DUMMY_SRC.as_bytes())
             .unwrap();
 
-        let link = Link::new("Chapter 1", chapter_path, None);
+        let link = Link::new("Chapter 1", chapter_path);
 
         (link, temp)
     }
@@ -339,7 +336,7 @@ And here is some \
             .write_all(b"Hello World!")
             .unwrap();
 
-        let mut second = Link::new("Nested Chapter 1", &second_path, None);
+        let mut second = Link::new("Nested Chapter 1", &second_path);
         second.number = Some(SectionNumber(vec![1, 2]));
 
         root.nested_items.push(second.clone().into());
@@ -365,7 +362,7 @@ And here is some \
 
     #[test]
     fn cant_load_a_nonexistent_chapter() {
-        let link = Link::new("Chapter 1", "/foo/bar/baz.md", None);
+        let link = Link::new("Chapter 1", "/foo/bar/baz.md");
 
         let got = load_chapter(&link, "", Vec::new());
         assert!(got.is_err());
@@ -382,7 +379,6 @@ And here is some \
             path: PathBuf::from("second.md"),
             parent_names: vec![String::from("Chapter 1")],
             sub_items: Vec::new(),
-            anchor: None,
         };
         let should_be = BookItem::Chapter(Chapter {
             name: String::from("Chapter 1"),
@@ -395,7 +391,6 @@ And here is some \
                 BookItem::Separator,
                 BookItem::Chapter(nested.clone()),
             ],
-            anchor: None,
         });
 
         let got = load_summary_item(&SummaryItem::Link(root), temp.path(), Vec::new()).unwrap();
@@ -470,7 +465,6 @@ And here is some \
                             Vec::new(),
                         )),
                     ],
-                    anchor: None,
                 }),
                 BookItem::Separator,
             ],
@@ -523,7 +517,6 @@ And here is some \
                             Vec::new(),
                         )),
                     ],
-                    anchor: None,
                 }),
                 BookItem::Separator,
             ],

src/book/summary.rs

@@ -72,8 +72,6 @@ pub struct Link {
     /// The location of the chapter's source file, taking the book's `src`
     /// directory as the root.
     pub location: PathBuf,
-    /// An optional anchor in the original link.
-    pub anchor: Option<String>,
     /// The section number, if this chapter is in the numbered section.
     pub number: Option<SectionNumber>,
     /// Any nested items this chapter may contain.
@@ -82,15 +80,10 @@ pub struct Link {
 
 impl Link {
     /// Create a new link with no nested items.
-    pub fn new<S: Into<String>, P: AsRef<Path>>(
+    pub fn new<S: Into<String>, P: AsRef<Path>>(name: S, location: P) -> Link {
-        name: S,
-        location: P,
-        anchor: Option<String>,
-    ) -> Link {
         Link {
             name: name.into(),
             location: location.as_ref().to_path_buf(),
-            anchor,
             number: None,
             nested_items: Vec::new(),
         }
@@ -102,7 +95,6 @@ impl Default for Link {
         Link {
             name: String::new(),
             location: PathBuf::new(),
-            anchor: None,
             number: None,
             nested_items: Vec::new(),
         }
@@ -284,18 +276,15 @@ impl<'a> SummaryParser<'a> {
         let link_content = collect_events!(self.stream, end Tag::Link(..));
         let name = stringify_events(link_content);
 
-        let mut split = href.splitn(2, '#');
+        if href.is_empty() {
-        let (href, anchor) = (split.next(), split.next());
+            Err(self.parse_error("You can't have an empty link."))
-
+        } else {
-        match href {
+            Ok(Link {
-            Some(href) if !href.is_empty() => Ok(Link {
                 name,
                 location: PathBuf::from(href.to_string()),
-                anchor: anchor.map(String::from),
                 number: None,
                 nested_items: Vec::new(),
-            }),
+            })
-            _ => Err(self.parse_error("You can't have an empty link.")),
         }
     }
 
@@ -687,12 +676,10 @@ mod tests {
             SummaryItem::Link(Link {
                 name: String::from("First"),
                 location: PathBuf::from("./first.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![1])),
                 nested_items: vec![SummaryItem::Link(Link {
                     name: String::from("Nested"),
                     location: PathBuf::from("./nested.md"),
-                    anchor: None,
                     number: Some(SectionNumber(vec![1, 1])),
                     nested_items: Vec::new(),
                 })],
@@ -700,7 +687,6 @@ mod tests {
             SummaryItem::Link(Link {
                 name: String::from("Second"),
                 location: PathBuf::from("./second.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![2])),
                 nested_items: Vec::new(),
             }),
@@ -722,14 +708,12 @@ mod tests {
             SummaryItem::Link(Link {
                 name: String::from("First"),
                 location: PathBuf::from("./first.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![1])),
                 nested_items: Vec::new(),
             }),
             SummaryItem::Link(Link {
                 name: String::from("Second"),
                 location: PathBuf::from("./second.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![2])),
                 nested_items: Vec::new(),
             }),
@@ -743,26 +727,6 @@ mod tests {
         assert_eq!(got, should_be);
     }
 
-    #[test]
-    fn parse_anchors() {
-        let src = "- [Link to anchor](./page.md#Foo)";
-
-        let should_be = vec![SummaryItem::Link(Link {
-            name: String::from("Link to anchor"),
-            location: PathBuf::from("./page.md"),
-            anchor: Some("Foo".to_string()),
-            number: Some(SectionNumber(vec![1])),
-            nested_items: Vec::new(),
-        })];
-
-        let mut parser = SummaryParser::new(src);
-        let _ = parser.stream.next();
-
-        let got = parser.parse_numbered().unwrap();
-
-        assert_eq!(got, should_be);
-    }
-
     /// This test ensures the book will continue to pass because it breaks the
     /// `SUMMARY.md` up using level 2 headers ([example]).
     ///
@@ -774,14 +738,12 @@ mod tests {
             SummaryItem::Link(Link {
                 name: String::from("First"),
                 location: PathBuf::from("./first.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![1])),
                 nested_items: Vec::new(),
             }),
             SummaryItem::Link(Link {
                 name: String::from("Second"),
                 location: PathBuf::from("./second.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![2])),
                 nested_items: Vec::new(),
             }),
@@ -797,13 +759,12 @@ mod tests {
 
     #[test]
     fn an_empty_link_location_is_an_error() {
-        for src in &["- [Empty]()\n", "- [Empty](#Foo)\n"] {
+        let src = "- [Empty]()\n";
-            let mut parser = SummaryParser::new(src);
+        let mut parser = SummaryParser::new(src);
-            parser.stream.next();
+        parser.stream.next();
 
-            let got = parser.parse_numbered();
+        let got = parser.parse_numbered();
-            assert!(got.is_err());
+        assert!(got.is_err());
-        }
     }
 
     /// Regression test for https://github.com/rust-lang/mdBook/issues/779
@@ -818,7 +779,6 @@ mod tests {
                 location: PathBuf::from("./first.md"),
                 number: Some(SectionNumber(vec![1])),
                 nested_items: Vec::new(),
-                anchor: None,
             }),
             SummaryItem::Separator,
             SummaryItem::Link(Link {
@@ -826,13 +786,11 @@ mod tests {
                 location: PathBuf::from("./second.md"),
                 number: Some(SectionNumber(vec![2])),
                 nested_items: Vec::new(),
-                anchor: None,
             }),
             SummaryItem::Separator,
             SummaryItem::Link(Link {
                 name: String::from("Third"),
                 location: PathBuf::from("./third.md"),
-                anchor: None,
                 number: Some(SectionNumber(vec![3])),
                 nested_items: Vec::new(),
             }),

src/renderer/html_handlebars/hbs_renderer.rs

@@ -31,7 +31,8 @@ impl HtmlHandlebars {
     ) -> Result<()> {
         // FIXME: This should be made DRY-er and rely less on mutable state
         if let BookItem::Chapter(ref ch) = *item {
-            let content = utils::render_markdown(&ch.content, ctx.html_config.curly_quotes);
+            let content = ch.content.clone();
+            let content = utils::render_markdown(&content, ctx.html_config.curly_quotes);
 
             let fixed_content = utils::render_markdown_with_path(
                 &ch.content,
@@ -80,10 +81,6 @@ impl HtmlHandlebars {
                     .insert("section".to_owned(), json!(section.to_string()));
             }
 
-            if let Some(anchor) = &ch.anchor {
-                ctx.data.insert("anchor".to_owned(), json!(anchor));
-            }
-
             // Render the handlebars template with the data
             debug!("Render template");
             let rendered = ctx.handlebars.render("index", &ctx.data)?;
@@ -523,10 +520,6 @@ fn make_data(
                     .to_str()
                     .chain_err(|| "Could not convert path to str")?;
                 chapter.insert("path".to_owned(), json!(path));
-
-                if let Some(anchor) = &ch.anchor {
-                    chapter.insert("anchor".to_owned(), json!(anchor));
-                }
             }
             BookItem::Separator => {
                 chapter.insert("spacer".to_owned(), json!("_spacer_"));

src/renderer/html_handlebars/helpers/navigation.rs

@@ -72,8 +72,8 @@ fn find_chapter(
             Target::Next => match chapters
                 .iter()
                 .filter(|chapter| {
-                    // Skip things like "spacer" or sub-links
+                    // Skip things like "spacer"
-                    chapter.contains_key("path") && !chapter.contains_key("anchor")
+                    chapter.contains_key("path")
                 })
                 .skip(1)
                 .next()
@@ -90,7 +90,7 @@ fn find_chapter(
 
     for item in chapters {
         match item.get("path") {
-            Some(path) if !path.is_empty() && item.get("anchor").is_none() => {
+            Some(path) if !path.is_empty() => {
                 if let Some(previous) = previous {
                     if let Some(item) = target.find(&base_path, &path, &item, &previous)? {
                         return Ok(Some(item));

src/renderer/html_handlebars/helpers/toc.rs

@@ -111,7 +111,7 @@ impl HelperDef for RenderToc {
                 if !path.is_empty() {
                     out.write("<a href=\"")?;
 
-                    let tmp = Path::new(path)
+                    let tmp = Path::new(item.get("path").expect("Error: path should be Some(_)"))
                         .with_extension("html")
                         .to_str()
                         .unwrap()
@@ -121,16 +121,9 @@ impl HelperDef for RenderToc {
                     // Add link
                     out.write(&utils::fs::path_to_root(&current_path))?;
                     out.write(&tmp)?;
-
-                    let anchor = item.get("anchor");
-                    if let Some(anchor) = anchor {
-                        out.write("#")?;
-                        out.write(anchor)?;
-                    }
-
                     out.write("\"")?;
 
-                    if anchor.is_none() && path == &current_path {
+                    if path == &current_path {
                         out.write(" class=\"active\"")?;
                     }