INI, Text File Reader...

[zehd]

If so and you haven't already called WriteTo, you can just set Tv = null to release it.

I'll try this

Edit: works...

[whurlston]

The reason I wanted this was, when I saw an error, O wanted to be able to rewrite the xmltv file using other code,but xmltv.dll won't release the file unless it was writtenout...

Ah. You mean the file remains locked and you can't write to it from another process. That's a bug in my code then. It should unlock the file as soon as the ReadFrom() process completes. I'll double check my code.

One thing though, if you write to the file from a separate process, the data that you have already loaded into your Tv object will not show the new information until you have it read the file again.

And one more thing... don't know if you can see this on the foum but accented characters are getting messed...

I'm losing accented characters...

That shouldn't be happening with that encoding set. I'll look into that too.

[whurlston]

The Birtles xmltv file is not quite xmltv compliant it seems. Specifically, the "generator-info-name" and "generator-info-url" attributes in the <tv> tag are shown as "generator-name" and "generator-url" instead so these tags will not get read. I can add a work around for this I think.

But that's not what's causing the deserialization failure, it's the date attribute in the <tv> tag. I took the format from the xmltv na_dd grabber but apparently I need to be more lenient in the format. I'll fix this as well.

As for the time offset issue, I was not properly converting to UTC time. I've corrected this for the next build.

I've also fixed an issue where <previously-shown /> would actually get output as <previously-shown date="00010101000000">

[zehd]

The Birtles xmltv file is not quite xmltv compliant it seems. Specifically, the "generator-info-name" and "generator-info-url" attributes in the <tv> tag are shown as "generator-name" and "generator-url" instead so these tags will not get read. I can add a work around for this I think.

But that's not what's causing the deserialization failure, it's the date attribute in the <tv> tag. I took the format from the xmltv na_dd grabber but apparently I need to be more lenient in the format. I'll fix this as well.

As for the time offset issue, I was not properly converting to UTC time. I've corrected this for the next build.

I've also fixed an issue where <previously-shown /> would actually get output as <previously-shown date="00010101000000">

All sounds good... When I was writing VB6-Ultra my way, I was opening and reading the file line by line as text and deciding what the tag was etc... So I could open anything and just adjust the parsing. That's why it was necessary to run a quick "is this DTD compliant?" check, which I still haven't found as easy a way as I have in VB6. But this check allowed for the lesser strict files...

According to the DTD, you don't need offset times in Programme.Start tags, and it shouldn't matter what is in the opening message <xml... generator etc...>

I'm seeing how your code works though, and it does a fair amount of strict assumption. Maybe I should make available to you all my test xmltv.xml I've gathered from around the world...

[whurlston]

Maybe I should make available to you all my test xmltv.xml I've gathered from around the world...

Just email me any files that you run into problems with and I'll fix it. I'll have a new build for you later tonight with the updates we've discussed.

[whurlston]

According to the DTD, you don't need offset times in Programme.Start tags, and it shouldn't matter what is in the opening message <xml... generator etc...>

You are right that the DTD is pretty lenient about the date but it specifically defines the attribute names for the tv tag:

Code:

121 <!ATTLIST tv date   CDATA #IMPLIED
  122              source-info-url     CDATA #IMPLIED
  123              source-info-name    CDATA #IMPLIED
  124              source-data-url     CDATA #IMPLIED
  125              generator-info-name CDATA #IMPLIED
  126              generator-info-url  CDATA #IMPLIED >

The data in those attributes of course can be anything.

[whurlston]

Ok, a quick test of the following code:

Code:

Dim ppTitles As Long = 0
Dim test As New XmlTv.Tv(originalFile, True)
test.AddSecondaryTitles("Blah Blah", True, ppTitles)
For Each p As XmlTv.Programme In test.Programmes
    If Not p.IsPaidProgramming Then
        p.Description_0 = (p.Title(0).Text & ". ") + p.Description_0
    End If
Next
test.WriteTo(newFile, True)

produced the following results:

Code:

Program Count: 27908
Paid Count: 3963
File read time: 3.315 seconds
Processing Time: 0.463 seconds
File Write Time: 3.806 seconds
Total Time: 7.584 seconds

Basically, I changed the secondary title on all Paid Programming then I added the program title to start of the description on everything that wasn't paid programming.

I'll do a final test tomorrow making sure I got everything we talked about and then send you the build.

[zehd]

Ok, a quick test of the following code:

Code:

Dim ppTitles As Long = 0
Dim test As New XmlTv.Tv(originalFile, True)
test.AddSecondaryTitles("Blah Blah", True, ppTitles)
For Each p As XmlTv.Programme In test.Programmes
    If Not p.IsPaidProgramming Then
        p.Description_0 = (p.Title(0).Text & ". ") + p.Description_0
    End If
Next
test.WriteTo(newFile, True)

produced the following results:

Code:

Program Count: 27908
Paid Count: 3963
File read time: 3.315 seconds
Processing Time: 0.463 seconds
File Write Time: 3.806 seconds
Total Time: 7.584 seconds

Basically, I changed the secondary title on all Paid Programming then I added the program title to start of the description on everything that wasn't paid programming.

I'll do a final test tomorrow making sure I got everything we talked about and then send you the build.

You timed the read, process and write, and the process was only .4 seconds?

[whurlston]

You timed the read, process and write, and the process was only .4 seconds?

Yup. Process time was the time it took to add the secondary titles to the Paid Programming and add the titles to the description on everything else. Reads and writes actually took longer than the processing. Now I have a fairly fast machine but it still gives you an idea of the possibilities.

[whurlston]

Try this dll. It is more lenient on date formats. If it hits a date format that it doesn't recognize, it will throw an error. Just let me know if it happens and I'll add any new formats you need.

You can see from my above example how to set the paid programming secondary titles and also there are a couple of new fields in the Programmes:

Description_0
Category_0
SecondaryTitle_0

These will give you direct access to p.Description(0).Text, etc. On gets, it will return an empty string if it doesn't exist and on sets, it will create it if it doesn't exist. This way, you don't need to do any checking on them (see above example again).

There are also some new bools like p.HasCategories, p.HasDescriptions, etc. I've also added a method to write the xml file without writing out the DTD since you shouldn't really need it anymore. Writes and reads are now better optimized and should no longer keep a lock on the file once the read/write process completes.

I've documented most of the new stuff so when you click on them in your Object Browser, it will tell you what it does.