Installing multiple copies of a theme that uses a data-template to modify user cards/profiles breaks both

[Firepup650 (Firepup Sixfifty)]

I was working on setting up my fork of the Custom Profile Link Component, and I needed two for different site links. However, currently, only users who have a username set for both sites show up. How can I ensure that both show up regardless of whether the other copy works or not?

[Firepup650 (Firepup Sixfifty)]

Honestly, it might not be the data-template, but that’s what I guessed it might be from looking at it for a bit.

[Lilly]

I’m not fully grasping what you are doing but perhaps it’s because you are writing to the same handlebars template twice rather than once (with two fields)?

[Firepup650 (Firepup Sixfifty)]

@Lilly

perhaps it’s because you are writing to the same handlebars template twice rather than once (with two fields)?

That was my suspicion as well, but I’m not sure how to work around that.

[Lilly]

can you not use one template? should be able to setup two fields on it, unless I am misunderstanding what you are trying to do. Did you get one link field to work? If you did, that’s great

[Firepup650 (Firepup Sixfifty)]

@Lilly

can you not use one template?

One of the templates is for the user card, the other is for the user profile, so I don’t think so.

@Lilly

Did you get one link field to work? If you did, that’s great

It works when one of two conditions are met:

1. You have two copies, two fields and the user you’re viewing has both names filled out
2. You have one copy, one field, and the user you’re viewing has the name filled out

[Lilly]

@Firepup650

One of the templates is for the user card, the other is for the user profile, so I don’t think so.

oh ok, I misunderstood, I figured you were putting 2 different links on the user card. Now it makes sense. yea two templates.

[Firepup650 (Firepup Sixfifty)]

@Lilly

I figured you were putting 2 different links on the user card.

I am, in a way, since I’m using one per copy of the Theme Component, which I suspect to be the root of the problem. I could be wrong about that though.

[Lilly]

I fed that chunk of code into Lola bot with no context and here was her reply:

@Lola bot

The JavaScript code you’ve shared seems to be in good shape, assuming that the settings.custom_profile_link_user_field_id and settings.custom_profile_link_debug_mode are properly defined elsewhere in your code.

This code registers two connector classes for the ‘user-profile-primary’ and ‘user-card-metadata’ hooks in Discourse. It then sets up a component for each, adding a customProfileLink property to the component if the necessary user fields are present.

The Handlebars templates are also correct, conditionally rendering links if customProfileLink is truthy.

However, without the context of the rest of your code, it’s hard to definitively say if there are any issues. If you’re experiencing any specific errors or unexpected behavior, those details would be helpful in diagnosing any potential problems.

[pfaffman (Jay Pfaffman)]

Are you saying that you forked the theme and then include your fork and the original one? You’ll need to change all of the references to it’s variables and the id passed to modify class.

[Firepup650 (Firepup Sixfifty)]

The original no longer functioned properly on Discourse, so I created a fork and revised it until it worked. At this time, I’ve installed two copies of my fork, which seem to be conflicting with each other.

[pfaffman (Jay Pfaffman)]

@Firepup650

, I’ve installed two copies of my fork, which seem to be conflicting with each other.

That’s what I’m saying. You’ll need to rename stuff so that it doesn’t conflict.

You likely want to make one theme do both things rather than having two copies.

[Firepup650 (Firepup Sixfifty)]

@pfaffman

That’s what I’m saying. You’ll need to rename stuff so that it doesn’t conflict.

How would I do that exactly? One thought I had was to add the TC’s ID to the handlebars, but I’m not sure how I would do that.

@pfaffman

You likely want to make one theme do both things rather than having two copies.

Wouldn’t that sort of break the original use-case of this theme component?

[pfaffman (Jay Pfaffman)]

@Firepup650

Wouldn’t that sort of break the original use-case of this theme component?

I don’t know, but I though it was broken already?

If it adds a link using a custom field and you want it to add two then you’d write it such that it could be configured to include any number of links to custom fields?

[Firepup650 (Firepup Sixfifty)]

@pfaffman

I don’t know, but I though it was broken already?

I made my fork to fix bugs with the original, so I’d hope mine isn’t broken.

@pfaffman

If it adds a link using a custom field and you want it to add two then you’d write it such that it could be configured to include any number of links to custom fields?

That could work, but I’m not sure how to do that.

[Firepup650 (Firepup Sixfifty)]

@Firepup650

@pfaffman

If it adds a link using a custom field and you want it to add two then you’d write it such that it could be configured to include any number of links to custom fields?

That could work, but I’m not sure how to do that.

I’ve started working on this. My first real roadblock here is processing multiple IDs. I think I would do it using a foreach loop (like I’ve done higher in the code), but I’m unsure how to do that in Handlebars.

[pfaffman (Jay Pfaffman)]

@Firepup650

I’m unsure how to do that in Handlebars

Sounds like you’re on the right path.

I can never remember and Google “handlebars for loop” and find examples.

[Firepup650 (Firepup Sixfifty)]

After fiddling with this for about an hour, I’m going to say I’m stumped. This code:

          {{#each this.customProfileLink as |links|}}
              <a href="{{link.[2]}}{{link.[0]}}" target="_blank">{{link.[1]}}</a>
              <p>{{link}}--{{link.content}}</p>
          {{/each}}

Feels like it should work, but everything I can find on #each seems to vary, and this doesn’t work. Anyone have any insight here?

And that one was doubly on me. Used links for the item, and link in the processor. And I didn’t check the user profile, only the user card (User profile was correct for some reason)