Skip to content
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.

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

Remove xLinkingBookDefs import from psnlYeeshaPageChanges.py #1664

Merged
merged 3 commits into from
Mar 2, 2025

Conversation

Hazado
Copy link
Contributor

@Hazado Hazado commented Feb 20, 2025

All the comments was doing here was list all the Relto pages you had if you had not gotten the 5th page
Removing that section allows removing the xLinkingBookDefs import

Copy link
Member

@Hoikas Hoikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really sure why we want to remove the xLinkingBookDefs import. But, we could probably salvage the log message with something like this:

# Top of file
import itertools

# Here:
PtDebugPrint("psnlYeeshaPageChanges: You've found the following Yeesha Pages:")
for thispage in itertools.count(1):
    varName = f"YeeshaPage{thispage}"
    if FoundValue := ageSDL.findVar(varName):
        PtDebugPrint(f"\t The previous value of the SDL variable '{varName}' is {FoundValue.getInt()}")
        if FoundValue.getInt() != 0: 
            PtDebugPrint(f"psnlYeeshaPageChanges: You have found Yeesha Page #{thispage}')
    else:
        # The first not found page variable is the end of the list of Yeesha Pages.
        break

Although, tbh, this should probably be in somewhere like Personal.py so we don't have to have the obnoxious guard hack.

@Hazado
Copy link
Contributor Author

Hazado commented Feb 28, 2025

Although, tbh, this should probably be in somewhere like Personal.py so we don't have to have the obnoxious guard hack.

Moved it to Personal.py instead

(02/28 10:44:49) Personal.ReportFoundPages(): You've found the following Yeesha Pages:
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage1' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage2' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage3' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage4' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage5' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage6' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage7' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage8' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage9' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage10' is 101
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage12' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage13' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage14' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage15' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage16' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage17' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage18' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage19' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage20' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage21' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage22' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage23' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage24' is 2
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage25' is 1
(02/28 10:44:49) The value of the SDL variable 'YeeshaPage26' is 2

@Hazado Hazado requested a review from Hoikas February 28, 2025 18:48
@Hoikas
Copy link
Member

Hoikas commented Mar 1, 2025

Did the snippet I post not work? What you've submitted still imports xLinkingBookDefs.

Copy link
Member

@Hoikas Hoikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one minor nitpick

@Hazado Hazado force-pushed the PruneComment branch 2 times, most recently from e6e2f60 to 898bad3 Compare March 2, 2025 01:42
Co-authored-by: Adam Johnson <[email protected]>
@Hoikas Hoikas merged commit 7a2ec88 into H-uru:master Mar 2, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants