2.3.0: New “Course Permalink Base”-feature is broken

  • Resolved Bernhard Kaindl

    (@bkaindl)


    10 months, 3 weeks ago

    The new feature to set the “Course Permalink Base” in 2.3.0 is not implemented fully:

    The new feature only sets the Course Permalink Base of new courses, but it does not update the Course Permalink Base of already created courses in the database:

    Hence, when you change the Course Permalink Base in the settings, existing courses will still have the Course URL Base of /courses/, not the one you’ve set, and thus, when to try to open the Course from Tutor Dashboard page, the new changed links in the Dashboard will point to nowhere and users get a 404 page.

    The page I need help with: [log in to see the link]

Viewing 4 replies - 1 through 4 (of 4 total)
  • largbasket

    (@largbasket)

    10 months, 3 weeks ago

    Dear @bkaindl

    We apologize for the inconvenience caused. In order to resolve this issue, we recommend updating your website’s permalink structure as per the instructions outlined in our documentation. After making the necessary changes, please remember to clear your browser cache.

    https://docs.themeum.com/tutor-lms/basic/permalink-settings/

    Thread Starter Bernhard Kaindl

    (@bkaindl)

    10 months, 3 weeks ago

    @largbasket: No, of course that does not help, I was aware of this needed setting. BTW, you should check that in the Tutor LMS settings, then you won’t get reports related to the regular permalink base structure of WordPress not matching the expectations of Tutor LMS.

    But this is a different case, and I am not a newbie. I’m a software developer.

    If you are not able to understand what I say, just reproduce it and please forward this to a developer, thanks!

    And it’s not a caching issue:

    All caching is disabled on my server, no CDN and here is the output:

    After having set the new feature for changing the Lesson permalink base (the same happens with the course permalink base as well), from “lessons” to “lesson”, I get this output:

    > wget –save-headers https://backup3.powerakademie.at/courses/2-lehrjahr-modul-2-die-macht-der-worte/lesson{s,}/lektionsentwurf/ -o-|grep -e HTTP -e cache
    > HTTP request sent, awaiting response… 200 OK
    > HTTP request sent, awaiting response… 404 Not Found

    As shown, despite no caching, the lesson is still served from “/lessons”, and not from the expected “/lesson”.

    Just try it yourself.

    Thread Starter Bernhard Kaindl

    (@bkaindl)

    10 months, 3 weeks ago

    Here is the same check now with the course base slug changed from “courses” to “course”:

    wget –save-headers https://backup3.powerakademie.at/course{s,}/2-lehrjahr-modul-2-die-macht-der-worte/lessons/lektionsentwurf/ -o-|grep -e HTTP -e cache
    HTTP request sent, awaiting response… 200 OK
    HTTP request sent, awaiting response… 404 Not Found

    The OK request was for /courses/ and the 404 was for /course/ (which was set).

    AFAICS, a fix, which was developed by Shaikh Ahmed Shewa who is the developer with the most contributions to Tutor LMS was overlooked by

    Avatar

    Md.Harun-Ur-Rashid when he released 2.3.0:

    https://github.com/themeum/tutor/pull/564

    Maybe it would be good if Md.Harun-Ur-Rashid would invite the other developers and especially Shaikh Ahmed Shewa for review when releasing a new version, and especially he shall also check to pull requests at

    https://github.com/themeum/tutor/pulls

    At this URL there are currently also number of fixes for Tutor LMS which should be applied to the Development version of Tutor LMS ASAP!

    Also, it would be good if both developers would check their GitHub notifications at https://github.com/notifications for new pull requests from other developers on a regular basis (or just (re-)enable their GitHub notification emails) and follow up on these Pull Requests.

    Moderator Jan Dembowski

    (@jdembowski)

    Forum Moderator and Brute Squad

    10 months, 3 weeks ago

    @bkaindl I archived your two identical topics about this problem. Please do not create duplicate topics again.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘2.3.0: New “Course Permalink Base”-feature is broken’ is closed to new replies.