Author Topic: New IMDb People v4 Script Discussion  (Read 63 times)

0 Members and 1 Guest are viewing this topic.

Offline afrocuban

  • Moderator
  • *****
  • Posts: 585
    • View Profile
New IMDb People v4 Script Discussion
« on: January 20, 2025, 02:14:19 am »
Hello Ivek.

Do you have an idea how could I force "BioList" from this function to populate to the field from the snippet in the file attached, while ShouldParseBio is set to 0 or is set to "Set If Empty" but bio field is not empty, on the last Exit from PVD?

BioList properly parses but it won't populate. The only trick I can do to populate BioList is to set "Bio" differently to "Overwrite" regardles of the state of the bio field, or to set it to "Set If Empty" if it was empty upon starting PVD with opBio=0 in the pvdconfig.ini (from the last PVD exit).
« Last Edit: January 20, 2025, 02:43:38 am by afrocuban »

Offline afrocuban

  • Moderator
  • *****
  • Posts: 585
    • View Profile
Re: New IMDb People v4 Script Discussion
« Reply #1 on: January 20, 2025, 02:45:11 am »
Also,

I can't log anything and I don't know what this snippet is for:


            //~Filmography index~
            If Pos('<h3 class="ipc-title__text"><span>Credits</span></h3>',CombinedHTML)>0 Then Begin
                curPos:=Pos('<h3 class="ipc-title__text"><span>Credits</span></h3>',CombinedHTML);
                If curPos>0 Then Begin
                    EndPos:=curPos;
                    ItemValueBase:=HTMLValues(CombinedHTML,'<h3 class="ipc-title__text"><span>Credits</span></h3>','<span class="ipc-chip__text">IMDbPro</span>','<span class="ipc-chip__text">','<span class="ipc-chip__count">',',  ',EndPos);             
                    If ItemValueBase <> '' then ItemListBase:=ItemListBase+ #13#10 +'Additional Filmography Career:  '+ItemValueBase+'  ';
         LogMessage('      Parse Results Filmography index:'+ItemValueBase+'||');
      End;
   End;
   
What it is supposed to catch?

Offline Ivek23

  • Global Moderator
  • *****
  • Posts: 2778
    • View Profile
Re: New IMDb People v4 Script Discussion
« Reply #2 on: January 20, 2025, 08:38:22 am »
Add all the missing bio codes (ShouldParseBio and similar) that you didn't add to the text file so I can run tests, only then will I see what's happening and maybe find a way to add information to the bio field.
Ivek23
Win 10 64bit (32bit)   PVD v0.9.9.21, PVD v1.0.2.7, PVD v1.0.2.7 + MOD


Offline afrocuban

  • Moderator
  • *****
  • Posts: 585
    • View Profile
Re: New IMDb People v4 Script Discussion
« Reply #3 on: January 21, 2025, 01:35:45 am »
Thanks a lot. If I give it to you in this phase it might confuse me because I'm not a coder. So, I suggest to get back to this issue when I release v4 of the scripts.

Offline Ivek23

  • Global Moderator
  • *****
  • Posts: 2778
    • View Profile
Re: New IMDb People v4 Script Discussion
« Reply #4 on: January 22, 2025, 12:13:54 pm »
         - Job Title, Career and other personal data now moved from comment to bio field.

Job title, career and almost all other personal information do not belong in the bio field. Or rather, only Mini Biography information belongs in the bio field, which has been the case since the very beginning, when there was still an IMDb People plugin. With the IMDb People script, I added all other information to the comment field. Information from the IMDb People Biography page was added to the bio field, such as Name, Date of birth and death, Place of birth and death, Height and family information. This was visible or there was an option not to see it.

Otherwise, I don't use IMDb People scripts at all anymore, because I don't need IMDb People information at all, because it takes up too much space in my databases, which I use. For the last 10 years, I have been using the IMDb People script only for testing on old IMDb People databases.

So much about this from my side.
Ivek23
Win 10 64bit (32bit)   PVD v0.9.9.21, PVD v1.0.2.7, PVD v1.0.2.7 + MOD


Offline afrocuban

  • Moderator
  • *****
  • Posts: 585
    • View Profile
Re: New IMDb People v4 Script Discussion
« Reply #5 on: January 22, 2025, 07:58:00 pm »
I understand. Thank you. It will be anyway easy to customize script to move the data back to comment field. It will be a matter of simple cut/paste only.

I too started with People script because I knew it is not used that much, so i could learn along the way. As you already know many solutions from People script are almost copy/paste to other IMDb scripts. So it will be easier for me to work on other scripts. I don't tend them to be "official" scripts. I kindly offer them as a community contribution. I want to learn because I want PVD to be alive for me at least.