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

confusion between NAME and name (software name, computer name) in ms_multi_search.php #47

Closed
benkohler opened this issue Feb 18, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@benkohler
Copy link

Commit 05a1420 changed line ~795 as follows, to attempt to fix the confusion between "Software Name" and "Computer Name":

  •                   'NAME'=>'h.NAME',
    
    •                  //'NAME'=>'h.NAME',
      
    •                  $l->g(23)=>'h.name',
      

The change from h.NAME to h.name fixed the confusion, but after this fix the "Computer" column entries do not have a hyperlink to the specific machine entry.

Then commit be601e9 changed the line as follows, to fix the hyperlinking:

  •                  $l->g(23)=>'h.name',
    
  •                  'NAME'=>'h.NAME',
    

..but this undid the NAME/name change from earlier. I believe what this line needs to be, is:

'NAME'=>'h.name',

which is something of a combination of the previous 2 fixes. I have tested this locally and it seems to give the desired result.

@benkohler
Copy link
Author

I'm trying to figure out how changing h.NAME to h.name could possibly change the behavior like this-- is it possible that h.NAME is somehow getting replaced by the value of a.NAME somewhere else in this file?

@StCyr
Copy link
Contributor

StCyr commented Feb 19, 2016

Hello @benkohler

I've tried what you propose and just changed 'NAME'=>'h.NAME' to 'NAME'=>'h.name', and hyperlink still works indeed.

Also, the tables obtained when using the "All softwares" search seem also OK.

Thanks,

Cyrille

@gillesdubois gillesdubois added this to the 2.2 milestone Feb 26, 2016
@gillesdubois gillesdubois self-assigned this Feb 26, 2016
@xmun2k
Copy link

xmun2k commented Apr 12, 2016

There is a similar issue with Software Search feature. If a search is executed for a specific software title, in the results, the entries for "Computers" and "Software: Name" both list the same thing i.e. the computer name. Not sure if this should be listed as a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants