OMG

Bring back RegisterTrophy

Bug #620762 reported by Bart van Strien
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OMG
New
Undecided
Unassigned

Bug Description

Of course I understand you're still working on the API, but I must say that parsing a trophy object is hard enough, let alone making one, that's why I prefer RegisterTrophy over InsertTrophy.
Well that, and the fact that it is one of the few documented functions.

Revision history for this message
Seif Lotfy (seif) wrote : Re: [Bug 620762] [NEW] Bring back RegisterTrophy

The client doesnt need to parse a trophy file...
The client dumps it in a folder so that omg catches on it. the Client that
throws it in just needs to know the ID of the Trophy so he can unlock it
over dbus on omg :)

On Fri, Aug 20, 2010 at 12:52 AM, bartbes <email address hidden> wrote:

> Public bug reported:
>
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
> ** Affects: omg
> Importance: Undecided
> Status: New
>
> --
> Bring back RegisterTrophy
> https://bugs.launchpad.net/bugs/620762
> You received this bug notification because you are a member of OMG!,
> which is subscribed to omg.
>
> Status in OMG!: New
>
> Bug description:
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
>
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Bart van Strien (bartbes) wrote :

I'd like to say you still need to parse a trophy *object* it for GetTrophies.

Anyway, back on-topic:
So, you deprecate one of the 4 documented functions.. well, okay, I can manage, but what is the file format? The 'dummy' file was filled with 'random' strings..

Revision history for this message
Seif Lotfy (seif) wrote : Re: [Bug 620762] Re: Bring back RegisterTrophy

The client never parses
all he needs to know is the tophy id...
the daemon itself does all the parsing :)

On Fri, Aug 20, 2010 at 11:50 AM, bartbes <email address hidden> wrote:

> I'd like to say you still need to parse a trophy *object* it for
> GetTrophies.
>
> Anyway, back on-topic:
> So, you deprecate one of the 4 documented functions.. well, okay, I can
> manage, but what is the file format? The 'dummy' file was filled with
> 'random' strings..
>
> --
> Bring back RegisterTrophy
> https://bugs.launchpad.net/bugs/620762
> You received this bug notification because you are a member of OMG!,
> which is subscribed to omg.
>
> Status in OMG!: New
>
> Bug description:
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
>
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Bart van Strien (bartbes) wrote :

You kind of need to create the file with the trophies... don't you?

Revision history for this message
Seif Lotfy (seif) wrote :

yeah developers create it by hand.. its easy
If you are going to develop a trophy you develop it by hand and
ofcourse remember the id ur goign to give it
then when in ur app the conditons for reqrding the trophy is met u signal
AwardTrophy with the according ID

On Fri, Aug 20, 2010 at 1:58 PM, bartbes <email address hidden> wrote:

> You kind of need to create the file with the trophies... don't you?
>
> --
> Bring back RegisterTrophy
> https://bugs.launchpad.net/bugs/620762
> You received this bug notification because you are a member of OMG!,
> which is subscribed to omg.
>
> Status in OMG!: New
>
> Bug description:
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
>
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Bart van Strien (bartbes) wrote :

And that's why I'm asking what the format is, or at least a better documentation of it.

Revision history for this message
Bart van Strien (bartbes) wrote :

Also, what was wrong with RegisterTrophy?

Revision history for this message
Seif Lotfy (seif) wrote :

yeah will write that out soon
its simple actually

[Trophy]
Title =
Description =
Application =
Set =
IconPath =
On Fri, Aug 20, 2010 at 2:35 PM, bartbes <email address hidden> wrote:

> And that's why I'm asking what the format is, or at least a better
> documentation of it.
>
> --
> Bring back RegisterTrophy
> https://bugs.launchpad.net/bugs/620762
> You received this bug notification because you are a member of OMG!,
> which is subscribed to omg.
>
> Status in OMG!: New
>
> Bug description:
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
>
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Seif Lotfy (seif) wrote :

[Trophy]
Title = Tomboy Novice
Description = You have one Tomboy Note
Application = Tomboy
Set = Novice Set
IconPath = tomboy.png
Priortiy = High
Locale =

On Fri, Aug 20, 2010 at 2:49 PM, Seif Lotfy <email address hidden> wrote:

> yeah will write that out soon
> its simple actually
>
> [Trophy]
> Title =
> Description =
> Application =
> Set =
> IconPath =
>
> On Fri, Aug 20, 2010 at 2:35 PM, bartbes <email address hidden> wrote:
>
>> And that's why I'm asking what the format is, or at least a better
>> documentation of it.
>>
>> --
>> Bring back RegisterTrophy
>> https://bugs.launchpad.net/bugs/620762
>> You received this bug notification because you are a member of OMG!,
>> which is subscribed to omg.
>>
>> Status in OMG!: New
>>
>> Bug description:
>> Of course I understand you're still working on the API, but I must say
>> that parsing a trophy object is hard enough, let alone making one, that's
>> why I prefer RegisterTrophy over InsertTrophy.
>> Well that, and the fact that it is one of the few documented functions.
>>
>>
>>
>
>
> --
> This is me doing some advertisement for my blog http://seilo.geekyogre.com
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Seif Lotfy (seif) wrote :

The Locale will be an issue

On Fri, Aug 20, 2010 at 2:35 PM, bartbes <email address hidden> wrote:

> Also, what was wrong with RegisterTrophy?
>
> --
> Bring back RegisterTrophy
> https://bugs.launchpad.net/bugs/620762
> You received this bug notification because you are a member of OMG!,
> which is subscribed to omg.
>
> Status in OMG!: New
>
> Bug description:
> Of course I understand you're still working on the API, but I must say that
> parsing a trophy object is hard enough, let alone making one, that's why I
> prefer RegisterTrophy over InsertTrophy.
> Well that, and the fact that it is one of the few documented functions.
>
>
>

--
This is me doing some advertisement for my blog http://seilo.geekyogre.com

Revision history for this message
Manish Sinha (मनीष सिन्हा) (manishsinha) wrote :

The format of the Trophy file is
It starts with something inside [ and ]

Then it has a list of Key=Value pair on each line. This is what the dameon looks for in the trophy file.

All the lines which have a "="in it is taken into consideration and rest all the rejected. This means you can even have
[I met my girlfriend today morning] in the first line

The allowed keys are
* ID
* Title
* Description
* Application
* SetName
* IconPath
* Priority
* Locale

If you use any other keys, they are ignored. Please dont use anything like

Id=Foo=Blah

it would also be rejected. It only looks for
Key=Value

All leading and trailing whitespaces are stripped out from Key and Value

this means
___Id___=__Me_too_
Where _ means whitespace

Finally this will become
Id and Me_too

Please don't put the values in quotes. You dont need to.

I think I have documented the Trophy format. I will move this to some place once we arrive at a conclusion that it is fine.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.