New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

papis-html: fixup and modernize code #60

Open

alexfikl wants to merge 9 commits into papis:master from alexfikl:papis-html-modernize

alexfikl commented Jul 22, 2023 •

edited

Loading

This modernizes the papis-html plugin a bit to use the latest things and work with flake8.
The commits are fairly self-contained, but generally this

Adds and fixes all flake8 errors
Updates jQuery, Bootstrap, bibtex-js
Updates the index.html to work with Bootstrap 5 and some cleanup.

NOTE: This does not touch the Vue application in there!

github-advanced-security bot found potential problems

View reviewed changes

papis-html/papis_html/data/js/bibtex_js.js

+                                      return moment(x["DATE"]).format("YYYY");
+                                  }
+                                  return "";
+                              }

Check notice

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (95% of all statements in

the enclosing function

have an explicit semicolon).

papis-html/papis_html/data/js/bibtex_js.js

-                      // Check if authors are formatted
+                      // Get conjunction if set in author
+                      conjunction = format.attr('conjunction') ? format.attr('conjunction') : ', and';

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable conjunction is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

-                                  // Check if value is empty
-                                  if (name[index] != "") {
-                                      $(this).html($(this).html() + name[index]);
+                      for (i = 0; i < searchLength; i++) {

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable i is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                          var fullName = $.grep(name.slice(0, 4), Boolean).join(" ");
+                          author.attr('class', fullName);
+                          var textBefore = false;
+                          for (j = 0, ele = author.find("span:not(a)"), len = ele.length; j < len; ++j) {

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable j is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                          var fullName = $.grep(name.slice(0, 4), Boolean).join(" ");
+                          author.attr('class', fullName);
+                          var textBefore = false;
+                          for (j = 0, ele = author.find("span:not(a)"), len = ele.length; j < len; ++j) {

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable ele is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                          var fullName = $.grep(name.slice(0, 4), Boolean).join(" ");
+                          author.attr('class', fullName);
+                          var textBefore = false;
+                          for (j = 0, ele = author.find("span:not(a)"), len = ele.length; j < len; ++j) {

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable len is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                  var displayTuples = [];
+                  if (bibtexField == "author") {
+                      $(".bibtexentry span.author").each(function(i, obj) {
+                          authors = $(this).children("span:not(.bibtex_js_conjunction)");

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable authors is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                  if (bibtexField == "author") {
+                      $(".bibtexentry span.author").each(function(i, obj) {
+                          authors = $(this).children("span:not(.bibtex_js_conjunction)");
+                          authorLength = authors.length;

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable authorLength is used like a local variable, but is missing a declaration.

papis-html/papis_html/data/js/bibtex_js.js

+                      }
+                  } else {
+                      $(".bibtexentry span." + bibtexField).each(function(i, obj) {
+                          arrayString = [$(this).text()];

Check warning

Code scanning / CodeQL

Missing variable declaration Warning

Variable arrayString is used like a local variable, but is missing a declaration.

alexfikl added 9 commits

July 22, 2023 20:12


          papis-html: add setup.cfg with flake8 configuration

9c948df


          papis-html: add flake8 develop dependencies

9237a0e


          ci: add flake8 linting

3f3077d


          papis-html: fix flake8 errors

5fae4db


          papis-html: bump js dependencies

6a21d0b


          papis-html: return when directory exists

5b37d20


          papis-html: update readme

7bfbbd1


          papis-html: update bootstrap versions

b8c0df4


          papis-html: update template to bootstrap 5

alexfikl force-pushed the papis-html-modernize branch from 5670bdd to 2194962 Compare

July 22, 2023 17:12

Author

alexfikl commented Jul 22, 2023

This is how the generated page looks now:

github-advanced-security bot found potential problems

View reviewed changes

papis-html/setup.py

Comment on lines +36 to +37

		"papis", "html", "bibtex", "javascript"
		"management", "cli", "biliography"

Check warning

Code scanning / CodeQL

Implicit string concatenation in a list Warning

Implicit string concatenation. Maybe missing a comma?

Author

alexfikl commented Jul 22, 2023

@alejandrogallo What do you think about updating this? It's very very simple compared to papis serve, but it's a nice example plugin if nothing else 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet