Sorting of plugin search results

[jennydaman]

ChRIS_store_ui/src/components/Plugins/Plugins.js

Line 218 in ca45de4

title="Sort By"

Some sorts could be:

• alphabetical, by author
• alphabetical, by name
• chronological, by date created

[sonylomo]

Can I work on this one @jennydaman?

[sonylomo]

@jennydaman, I'm trying to resolve conflicts but my code isn't passing these 2 tests. I didn't change any code related to them. I've checking what's wrong and updated my branch to be inline with the master before fixing plugin sort.

I've also checked if the "mark as favorite" functionality on plugins store website is working but it's not - was it working before?

[jennydaman]

Try these commands:

git remote add upstream git@github.com:FNNDSC/ChRIS_store_ui.git
git fetch upstream
git diff upstream/master

I found the bug:

diff --git a/src/components/Plugins/Plugins.js b/src/components/Plugins/Plugins.js
index a157f49..e5e2b45 100755
--- a/src/components/Plugins/Plugins.js
+++ b/src/components/Plugins/Plugins.js
@@ -113,20 +114,23 @@ export class Plugins extends Component {
     }
   }

-  fetchPlugins = () => {
-    const params = new URLSearchParams(window.location.search)
-    const name = params.get('q') //get value searched from the URL
+  fetchPlugins() {
+    const params = new URLSearchParams(window.location.search);
+    const name = params.get("q"); //get value searched from the URL
+
     const searchParams = {
       limit: 20,
       offset: 0,
       name_title_category:name,
-    };
+    }
+

Notice the declaration of fetchPlugins was changed. This is an oddity with syntax. If a function is defined as fetchPlugins() {, this will be undefined unless you manually "bind" it.

[sonylomo]

Notice the declaration of fetchPlugins was changed. This is an oddity with syntax. If a function is defined as fetchPlugins() {, this will be undefined unless you manually "bind" it.

Thanks for that @jennydaman. What about the starsByPlugin test?

[jennydaman]

@sonylomo all tests are passing on sonylomo@b154b30

[sonylomo]

Yeah, I spotted the problem with the tests. Everything is ok now.

[sonylomo]

I've closed the other PR because I deleted that forked repo 😁.